Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

flameshot: refresh page #8022

Merged
merged 7 commits into from
May 16, 2022
Merged

flameshot: refresh page #8022

merged 7 commits into from
May 16, 2022

Conversation

EmilyGraceSeville7cf
Copy link
Contributor

  • The page (if new), does not already exist in the repository.
  • The page is in the correct platform directory (common/, linux/, etc.)
  • The page has 8 or fewer examples.
  • The PR title conforms to the recommended templates.
  • The page follows the content guidelines.
  • The page description includes a link to documentation or a homepage (if applicable).

Version of the command being documented (if known): latest

EmilySeville7cfg added 4 commits April 16, 2022 13:18
Add mate-screenshot command page
f23cdac
Add --delay option example
33b48f0
Simplify descriptions
66f4193
Refresh page:
37c452d 
- reorder examples
- standardize with mate-screenshot
- more examples
@EmilyGraceSeville7cf EmilyGraceSeville7cf self-assigned this Apr 16, 2022
@github-actions github-actions bot added new command Issues requesting creation of a new page. page edit Changes to an existing page(s). labels Apr 16, 2022
@CleanMachine1
Copy link
Member

Another page seems to have escaped into here.

mfrw
mfrw reviewed Apr 18, 2022
View reviewed changes
pages/linux/flameshot.md Outdated Show resolved Hide resolved
@mfrw
Use the for a clipboard
d729dfa 
Co-authored-by: Muhammad Falak R Wani <falakreyaz@gmail.com>
mfrw
mfrw approved these changes Apr 20, 2022
View reviewed changes
Copy link
Member

@mfrw mfrw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@navarroaxel
Reduce screen number to 2
40a4a9e 
Co-authored-by: Axel Navarro <navarroaxel@gmail.com>
@navarroaxel navarroaxel merged commit cf51bfd into main May 16, 2022
@navarroaxel navarroaxel deleted the feature/refresh-flameshot branch May 16, 2022 12:35
@navarroaxel navarroaxel removed stalebot ignore new command Issues requesting creation of a new page. labels May 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CleanMachine1 CleanMachine1 CleanMachine1 left review comments

@navarroaxel navarroaxel navarroaxel approved these changes

@mfrw mfrw mfrw approved these changes

@einverne einverne Awaiting requested review from einverne

Assignees

@EmilyGraceSeville7cf EmilyGraceSeville7cf

Labels
page edit Changes to an existing page(s).
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@EmilyGraceSeville7cf @CleanMachine1 @navarroaxel @mfrw