f-spot 'create new tag' has double shortcuts assigned

Bug #516801 reported by sebastian-s
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
F-Spot
Fix Released
Low
One Hundred Papercuts
Fix Released
Low
Unassigned
f-spot (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

when calling up the 'Create new tag' dialogue in F-spot there are two buttons create and cancel which have the same keyboard shortcut assigned.

https://bugzilla.gnome.org/show_bug.cgi?id=608936

Related branches

Revision history for this message
sebastian-s (sebastian-s) wrote :
description: updated
Changed in f-spot:
importance: Undecided → Unknown
status: New → Unknown
Revision history for this message
Vish (vish) wrote :

Thanks for sending it upstream.

Changed in hundredpapercuts:
importance: Undecided → Low
milestone: none → lucid-round-7
status: New → Triaged
Revision history for this message
sebastian-s (sebastian-s) wrote :

@ Vish - I do what I can as I haven't dived into coding :-/

Revision history for this message
yves (yves-kurz-gmail) wrote :

I've created a patch and attached it to https://bugzilla.gnome.org/show_bug.cgi?id=608936.

Where should a patch be attached if a the same bug exist on lp and on bgo?

Revision history for this message
Sense Egbert Hofstede (sense) wrote :

Thank you for writing a patch! At the bottom of each bug report is a link called 'Add attachment or patch', you can use that to attach the patch.

Vish (vish)
Changed in f-spot (Ubuntu):
importance: Undecided → Low
status: New → Triaged
Revision history for this message
yves (yves-kurz-gmail) wrote :
Revision history for this message
yves (yves-kurz-gmail) wrote :

Does it make sense to add the patch to LP and BGO?

Revision history for this message
Sebastien Bacher (seb128) wrote :

it does make sense yes, thank you for working on those changes

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your work, I un-unsubscribe sponsors now since that breaks translations and it would be better to get it from upstream directly. I've pinged them for review. I will have a look at the f-spot hundredpapercut bugs before beta and upload changes if upstream didn't reply though

Revision history for this message
Vish (vish) wrote :

This bug has now been fixed upstream.

Changed in f-spot (Ubuntu):
status: Triaged → Fix Committed
Changed in hundredpapercuts:
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package f-spot - 0.6.1.5-2ubuntu1

---------------
f-spot (0.6.1.5-2ubuntu1) lucid; urgency=low

  * debian/patches/git_copy_attribute_normal.patch:
    - git change to set the copy attributes to normal, fix a write permission
      issue (lp: #483268)
  * debian/patches/git_no_urlencode.patch:
    - git change to not urlencode the mail subjects (lp: #454972)
  * debian/patches/git_correct_description.patch:
    - git change to fix the profiles description displayed (lp: #502744)
  * debian/patches/git_color_profile_printing_breakage.patch:
    - git change to fix printing when a color profile is selected (lp: #496134)
  * debian/patches/git_key_conflict.patch:
    - git change to fix a shortcut conflict (lp: #516801)
  * debian/patches/git_set_default_resize_value.patch:
    - git change to set a correct default resize value (lp: #513864)
  * debian/patches/ubuntu_edit_tag.patch:
    - change the edit tag label to be consistent with other entries
      (lp: #513452)
  * debian/patches/ubuntu_transition_timing.patch:
    - change by Alex Launi to make the transition effect between modes take 1s
      (lp: #513045)
  * debian/patches/ubuntu_slider_animation_tweaking.patch:
    - change by Alex Launi to reduce the slider animation timing (lp: #513004)
  * debian/patches/ubuntu_check_duplicate.patch:
    - change by Matt Perry to use a better label for the duplicates checking
      (lp: #511767)
 -- Sebastien Bacher <email address hidden> Mon, 01 Mar 2010 16:06:41 +0100

Changed in f-spot (Ubuntu):
status: Fix Committed → Fix Released
Vish (vish)
Changed in hundredpapercuts:
status: Fix Committed → Fix Released
Changed in f-spot:
importance: Unknown → Low
status: Unknown → 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.