After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 612701 - Unhandled mnemonic for "Find next" in Weather Preferences dialog
Unhandled mnemonic for "Find next" in Weather Preferences dialog
Status: RESOLVED FIXED
Product: gnome-applets
Classification: Other
Component: gweather
git master
Other All
: Normal normal
: ---
Assigned To: gnome-applets Maintainers
gnome-applets Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-03-12 14:01 UTC by Gianfranco Costamagna
Modified: 2010-03-15 16:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
typo (438.79 KB, image/png)
2010-03-12 14:01 UTC, Gianfranco Costamagna
Details
gweather 2.29.5 with patch (216.70 KB, application/octet-stream)
2010-03-15 15:17 UTC, Gianfranco Costamagna
Details

Description Gianfranco Costamagna 2010-03-12 14:01:46 UTC
Created attachment 155966 [details]
typo

(already reported inside bug 612647, just copying here)

Typo I found in gnome-applets

Step to see it:

Add "condizioni meteo" in gnome-panel
(/usr/lib/gnome-applets/gweather-applet-2)

Click on "preferenze" "località"

you will see "Trova s_uccessiva".

(How can I fix problems without asking you? It is so annoying and I can help in
translate something...)

/OT

[reply] [-] Comment 7 Milo Casagrande 2010-03-12 13:05:38 UTC

(In reply to comment #6)
> OFF TOPIC.
> 
> Another typo I found in gnome-applets

You should open different bugs for different projects.

> Step to see it:
> 
> Add "condizioni meteo" in gnome-panel
> (/usr/lib/gnome-applets/gweather-applet-2)
> 
> Click on "preferenze" "località"
> 
> you will see "Trova s_uccessiva".

What is the typo here?

> (How can I fix problems without asking you? It is so annoying and I can help in
> translate something...)

If you want helping out with translations, follow what's written here:
http://it.gnome.org/index.php/Traduzioni

You're helping even with the bug reports.

[reply] [-] Comment 8 Gianfranco Costamagna [reporter] 2010-03-12 13:12:23 UTC

Created an attachment (id=155961) [details]
error

[reply] [-] Comment 9 Gianfranco Costamagna [reporter] 2010-03-12 13:12:48 UTC

Please see my attachment, of course I can open another bug, but It is really
simple... :)

[reply] [-] Comment 10 Milo Casagrande 2010-03-12 13:47:52 UTC

That's probably not a typo, it could be a problem within the UI. Check with the
English version if problem persists.

These are the strings:
msgid "Find _Next"
msgstr "Trova s_uccessiva"

I know it's easy to write on a single bug, but it's definitely better to have
single task to handle.
Comment 1 Luca Ferretti 2010-03-12 14:51:57 UTC
It's a generic gweather issue, not related to Italian, translation. Reassigning and changing summary.
Comment 2 Luca Ferretti 2010-03-12 14:52:49 UTC
Confirming too...
Comment 3 André Klapper 2010-03-13 21:38:47 UTC
Code is in /gweather/gweather-pref.c:1270:
    pref->priv->find_next_btn = gtk_button_new_with_label (_("Find _Next"));

gtk_button_new_with_label() should probably be gtk_button_new_with_mnemonic().
Comment 4 Gianfranco Costamagna 2010-03-14 23:29:23 UTC
Does anybody have permissions to send this patch upstream?
I will build the plugin myself to see if it works!
Comment 5 Luca Ferretti 2010-03-15 00:55:51 UTC
I committed and pushed the change suggested by André due incoming hard code freeze :)
Comment 6 André Klapper 2010-03-15 10:36:46 UTC
Well, I was just guessing, so a verification that this fixes the issue is highly welcome. :-)

http://git.gnome.org/browse/gnome-applets/commit/?id=7ffb09cc2922e99259e86ee01bbccf8e2a99b07e
Comment 7 Luca Ferretti 2010-03-15 11:15:39 UTC
Unfortunately in my jhbuild sandbox gnome-panel is unable to load many applets, including weather :(
Comment 8 Gianfranco Costamagna 2010-03-15 15:17:56 UTC
Created attachment 156182 [details]
gweather 2.29.5 with patch
Comment 9 Gianfranco Costamagna 2010-03-15 15:20:29 UTC
Just built gweather from myself and it works...
I didn't use the patch, just downloaded the latest 2.29.5 gnome-applet and changed the line for printing the string "find next".
the "u" is underlined correctly.
(anyway I really don't know how to use the shortcut)

So I'll close this bug as fixed when I'll know how to use the shorcut to find the next item!


Thanks for your help

G.
Comment 10 André Klapper 2010-03-15 15:53:07 UTC
(In reply to comment #9)
> when I'll know how to use the shorcut to find the next item!

So what are the remaining issues with the shortcut if the U is now underlined?
Can you please be as exact as possible?
Comment 11 Gianfranco Costamagna 2010-03-15 16:34:06 UTC
The U is underlined, but with pressing "u" key I cannot get the next item...

I get it only when I press the "next item" key.

So, should I press another key like "alt" + "u" to get the button pressed by the hotkey?
Comment 12 Gianfranco Costamagna 2010-03-15 16:38:09 UTC
Now with the alt+u key is working...

So I'm closing this bug report as fixed.

Thanks to everybody.

Please reopen if the problem persists

Gianfranco