Add a quicklist entry for taking a screenshot of a selected area

Bug #952638 reported by Christopher Kyle Horton on 2012-03-12
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gnome-screenshot (Ubuntu)
Low
Unassigned

Bug Description

gnome-screenshot supports taking a screenshot of just a section of the screen, either as a selectable option in its window or by running gnome-screenshot -a in a terminal. It would be nice to also have this option available from its Unity quicklist, like the other two modes it already has (fullscreen and current window).

ProblemType: Bug
DistroRelease: Ubuntu 12.04
Package: gnome-screenshot 3.3.91-0ubuntu1
ProcVersionSignature: Ubuntu 3.2.0-18.29-generic 3.2.9
Uname: Linux 3.2.0-18-generic x86_64
ApportVersion: 1.94.1-0ubuntu2
Architecture: amd64
Date: Sun Mar 11 20:12:45 2012
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Alpha amd64 (20111129.1)
ProcEnviron:
 TERM=xterm
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: gnome-screenshot
UpgradeStatus: Upgraded to precise on 2012-03-11 (0 days ago)

I went and modified my /usr/share/applications/gnome-screenshot.desktop file to add quicklist support for this option, and it works on my own machine.

I'm still largely unfamiliar with how creating patches and the bug fixing process in general works, but after getting the source with apt-get source and working through the instructions for using edit-patch provided by https://wiki.ubuntu.com/PackagingGuide/Recipes/Debdiff, I think I now have a patch which can be submitted for consideration. (Let me know if I missed anything that I would still need to take care of.)

tags: added: patch

The attachment "02_add_selected_area_quicklist_option.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

After getting some feedback through Google+, I have updated the patch to use the new style instead of the X-Ayatana one.

Sebastien Bacher (seb128) wrote :

Thank you for your work, I fixed some issues in your patch and included it for the next upload

issues were:
- TargetEnvironment is deprecated in favor of OnlyShowIn
- you need to prefix Name with a _ so it's translatable
- the actions group needs a trailing ";"

I also fixed the capitalization to be consistent with other unity lists

Changed in gnome-screenshot (Ubuntu):
importance: Undecided → Low
status: New → Fix Committed

@Sebastien Thanks for catching my errors and providing feedback! Great to know it was still accepted.

Should this be marked Fix Released now, since the added functionality has appeared in an update a few days ago?

kroq-gar78 (kroq-gar78) on 2013-01-12
Changed in gnome-screenshot (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers