Merge lp:~paolorotolo/rhythmbox/fix-for-991107 into lp:rhythmbox

Proposed by Paolo Rotolo
Status: Needs review
Proposed branch: lp:~paolorotolo/rhythmbox/fix-for-991107
Merge into: lp:rhythmbox
Diff against target: 15 lines (+2/-2)
1 file modified
plugins/lyrics/lyrics.py (+2/-2)
To merge this branch: bzr merge lp:~paolorotolo/rhythmbox/fix-for-991107
Reviewer Review Type Date Requested Status
Marc Deslauriers (community) Needs Fixing
Jonathan Matthew Pending
Review via email: mp+104368@code.launchpad.net

Description of the change

Fix typo (LP #991107).

To post a comment you must log in.
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

NACK on the fix. The underscore is there so the next character becomes the accelerator key.
The right fix would be to create the ToggleButton with use_underline=True.

review: Needs Fixing

Unmerged revisions

7999. By Paolo Rotolo

  * plugins/lyrics/lyrics.py: fix typo. (LP #991107)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/lyrics/lyrics.py'
--- plugins/lyrics/lyrics.py 2012-02-01 21:25:55 +0000
+++ plugins/lyrics/lyrics.py 2012-05-02 11:42:18 +0000
@@ -224,9 +224,9 @@
224 self.get_lyrics()224 self.get_lyrics()
225 225
226226
227 self.edit = Gtk.ToggleButton(label=_("_Edit"))227 self.edit = Gtk.ToggleButton(label=_("Edit"))
228 self.edit.connect('toggled', edit_callback)228 self.edit.connect('toggled', edit_callback)
229 self.discard = Gtk.Button(label=_("_Search again"))229 self.discard = Gtk.Button(label=_("Search again"))
230 self.discard.connect('clicked', discard_callback)230 self.discard.connect('clicked', discard_callback)
231 self.clear = Gtk.Button.new_from_stock(Gtk.STOCK_CLEAR)231 self.clear = Gtk.Button.new_from_stock(Gtk.STOCK_CLEAR)
232 self.clear.connect('clicked', clear_callback)232 self.clear.connect('clicked', clear_callback)

Subscribers

People subscribed via source and target branches

to all changes: