Typo in button labels for Rhythmbox lyrics viewer

[Ben Linsey-Bloom]

Right click on any song in rhythmbox library.
Click on the lyrics tab.
The 'Edit' and 'Search Again' buttons have underscores in their labels.

12.04 en-gb localisation.

See screenshot attached.

[Paolo Rotolo (paolorotolo)]

Merge proposed.

[Marc Deslauriers (mdeslaur)]

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.

[Timothy Arceri (t-fridey)]

I've submitted a patch upstream

[Timothy Arceri (t-fridey)]

Fix has been committed upstream

[Chris Wilson (notgary-deactivatedaccount)]

Excellent, first patch of the R cycle. Congratulations :)

On 25 October 2012 22:25, Timothy Arceri <email address hidden> wrote:

> Fix has been committed upstream
>
> ** Changed in: hundredpapercuts
> Status: In Progress => Fix Committed
>
> ** Changed in: rhythmbox (Ubuntu)
> Status: Confirmed => Fix Committed
>
> --
> You received this bug notification because you are subscribed to One
> Hundred Paper Cuts.
> Matching subscriptions: Papercut bug mail
> https://bugs.launchpad.net/bugs/991107
>
> Title:
> Typo in button labels for Rhythmbox lyrics viewer
>
> Status in One Hundred Paper Cuts:
> Fix Committed
> Status in The Rhythmbox Music Management Application:
> New
> Status in “rhythmbox” package in Ubuntu:
> Fix Committed
>
> Bug description:
> Right click on any song in rhythmbox library.
> Click on the lyrics tab.
> The 'Edit' and 'Search Again' buttons have underscores in their labels.
>
> 12.04 en-gb localisation.
>
> See screenshot attached.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/hundredpapercuts/+bug/991107/+subscriptions
>

[Jeremy Bícha (jbicha)]

This has been fixed in Ubuntu 13.04.