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

Bug #952638 reported by Christopher Kyle Horton
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
gnome-screenshot (Ubuntu)
Fix Released
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)

Revision history for this message
Christopher Kyle Horton (christhehorton) wrote :
Revision history for this message
Christopher Kyle Horton (christhehorton) wrote :

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
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

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.]

Revision history for this message
Christopher Kyle Horton (christhehorton) wrote :

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

Revision history for this message
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
Revision history for this message
Christopher Kyle Horton (christhehorton) wrote :

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

Revision history for this message
Christopher Kyle Horton (christhehorton) wrote :

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

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

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.