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 628589 - Notification message list message is too long
Notification message list message is too long
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: i18n
2.31.x
Other All
: Normal normal
: 2.32.0
Assigned To: Orca Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-09-02 06:47 UTC by Hammer Attila
Modified: 2010-09-20 10:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
This patch possible fix this bug, please look. (1.02 KB, patch)
2010-09-02 07:00 UTC, Hammer Attila
none Details | Review
revision #1 (1.30 KB, patch)
2010-09-02 17:11 UTC, Jose Vilmar Estacio de Souza
committed Details | Review

Description Hammer Attila 2010-09-02 06:47:15 UTC
Dear Developers,

Now, following long message are presented in learning mode and Orca preferences/keybingings page with related to notification message list mode feature:
#. Translators: Orca can enable the notification messages
#. list mode when the user presses a shortcut key.
#.
#: ../src/orca/notification_messages.py:98
msgid ""
"Enable notification message list mode. Press Escape to exit or h for help"

This is a very long message. For example, the hungarian translation with this message are following:
msgstr ""
"Értesítés listázó mód engedélyezése. A kilépéshez nyomja meg az Escape "
"billentyűt, a segítségért a h billentyűt."

I attaching next comment a patch with shortest this message, if my english is not good, please correct.

Attila
Comment 1 Hammer Attila 2010-09-02 07:00:49 UTC
Created attachment 169321 [details] [review]
This patch possible fix this bug, please look.

Joanie, this patch shortest the long notification message list keybinging and learning mode message with a shortest message. If my english message is not good, please correct my message.

Attila
Comment 2 Joanmarie Diggs (IRC: joanie) 2010-09-02 07:33:54 UTC
Attila, thanks.

I'm not sure what to do here. Ale had pointed out this issue to me. The problem is that we need some way to be sure the user knows how to use and how to get out of this mode. Just like with Orca+H. So we left it as José had it.

CCing both Ale and José so they can chime in.
Comment 3 Hammer Attila 2010-09-02 08:38:03 UTC
Not possible to do similar way with keyboard shortcuts list feature message separation?
For example, my short message is enough with Orca keybindings page (need an absolute short message), but if the user press the user defined shortcut key, need present a longest message to notify the user how can use this feature. This feature is an absolute useful and wished feature, I am very happy when I see this feature.

For example, when the user activate this function, the original longest message is very good. If the user press the h key after this message, he or she see detailed informations (navigation keys instructions), this is absolute right and done.

Attila
Comment 4 Jose Vilmar Estacio de Souza 2010-09-02 14:32:30 UTC
(In reply to comment #3)
> Not possible to do similar way with keyboard shortcuts list feature message
> separation?
> For example, my short message is enough with Orca keybindings page (need an
> absolute short message), but if the user press the user defined shortcut key,
> need present a longest message to notify the user how can use this feature.
> This feature is an absolute useful and wished feature, I am very happy when I
> see this feature.
> 
> For example, when the user activate this function, the original longest message
> is very good. If the user press the h key after this message, he or she see
> detailed informations (navigation keys instructions), this is absolute right
> and done.
> 
> Attila
Hi.
When user press the key to enter in the list mode, a short message is spoken telling the number of messages, and instructing the user to press h for help.
How about showing the help instead of this short message?
Comment 5 Joanmarie Diggs (IRC: joanie) 2010-09-02 14:36:04 UTC
(In reply to comment #4)
> When user press the key to enter in the list mode, a short message is spoken
> telling the number of messages, and instructing the user to press h for help.
> How about showing the help instead of this short message?

And by doing so, not introduce any string changes for which I will have to ask for permission to break string freeze? That would have my vote. <smile>
Comment 6 Jose Vilmar Estacio de Souza 2010-09-02 14:55:49 UTC
(In reply to comment #5)
> (In reply to comment #4)
> > When user press the key to enter in the list mode, a short message is spoken
> > telling the number of messages, and instructing the user to press h for help.
> > How about showing the help instead of this short message?
> 
> And by doing so, not introduce any string changes for which I will have to ask
> for permission to break string freeze? That would have my vote. <smile>
Seems to me that we need to apply the patch attached in comment #2.
No more strings will be changed.
Comment 7 Hammer Attila 2010-09-02 16:33:58 UTC
Now already string freeze, or near it?
Jose, the problem only the long message is presenting the Enable notification message list mode. Press Escape to exit or h for help" with Orca preferences/keybindings page.

Need shorting this feature learning mode and Orca preferences/keybindings page message, and perhaps next developing cicle insert this long message when the user activate this feature. But if this is not good ydea, I accept of course.

Attila
Comment 8 Jose Vilmar Estacio de Souza 2010-09-02 17:11:24 UTC
Created attachment 169377 [details] [review]
revision #1

Implements Attila's suggestion.
Moreover, when the user enters the list mode, the help is spoken.
The help was already present in the scripts and I didn't need to include new strings.
Comment 9 Hammer Attila 2010-09-03 03:52:14 UTC
Jose, your patch is perfect I think, I tested.

Joanie, this fix method is good?

Attila
Comment 10 Joanmarie Diggs (IRC: joanie) 2010-09-03 08:15:58 UTC
This bit here looks like a new string as I do not see it amongst the current strings marked for translation in Orca:

+            _("Present notification messages list"))

If this is indeed a new string, getting that in would require that we request a freeze break.

Thoughts?
Comment 11 Hammer Attila 2010-09-03 15:36:06 UTC
Joanie, if possible, I accept request freeze break, because this is a need change. If this is impossible, I don't no how can possible fix this bug now with this next stable version.
Very need this message shorting if this is possible I think.

If this is impossible, because not accept the freeze break request, I think we possible fix this bug only next development version if anybody not have another ydea to fix this bug.
Comment 12 Joanmarie Diggs (IRC: joanie) 2010-09-03 16:31:56 UTC
Permission to break the string freeze requested.
Comment 13 Joanmarie Diggs (IRC: joanie) 2010-09-07 14:45:49 UTC
We received the requisite approvals. I will make this change later today.
Comment 14 Joanmarie Diggs (IRC: joanie) 2010-09-07 17:33:05 UTC
Review of attachment 169377 [details] [review]:

Committed. I will send the appropriate teams the announcement.

http://git.gnome.org/browse/orca/commit/?id=8ca3e9241ca1170a500907d56bbce7c15921b1cc