Rhythmbox

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

  1. fix-for-991107
  2. Merge into trunk
Proposed by Paolo Rotolo
Status: Needs review
Proposed branch: lp://staging/~paolorotolo/rhythmbox/fix-for-991107
Merge into: lp://staging/rhythmbox
Diff against target: 15 lines (+2/-2)
1 file modified
plugins/lyrics/lyrics.py (+2/-2)
To merge this branch: bzr merge lp://staging/~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.staging.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
Reply

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
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: