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 510642 - translation issues. helloho...
translation issues. helloho...
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: general
2.22.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
Depends on:
Blocks: 236276
 
 
Reported: 2008-01-19 17:19 UTC by André Klapper
Modified: 2008-08-28 14:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch for string changes (3.28 KB, patch)
2008-02-05 06:39 UTC, Suman Manjunath
none Details | Review
patch for string changes (3.29 KB, patch)
2008-02-05 10:03 UTC, Suman Manjunath
none Details | Review
patch for string changes (3.40 KB, patch)
2008-02-07 10:27 UTC, Suman Manjunath
committed Details | Review

Description André Klapper 2008-01-19 17:19:07 UTC
#: ../addressbook/addressbook.error.xml.h:21
"The address book {0} is read only, thus you cannot add contacts there. 
Select other address book, please."
#: ../calendar/calendar.error.xml.h:64
"You have a read-only calendar source selected. Change to Calendar View and 
highlight a calendar that can accept appointments."
please sync the style of these messages, the first one is not that helpful...


#: ../mail/mail-config.glade.h:2
"(Note: Requires restart)"
of computer? of app? (yes, the latter one.)


#: ../plugins/mail-notification/apps-evolution-mail-notification.schemas.in.h:2
"Blink icon in notification area."
#: ../plugins/mail-notification/apps-evolution-mail-notification.schemas.in.h:4
"Enable icon in notification area."
do i have to choose between the blink icon and the enable icon? or do you personally blink the icon for me? :)


#: ../mail/evolution-mail.schemas.in.h:107
"Show the email of the sender in the messages composite column in the message list."
srini, what is a composite column? i have NEVER heard that before.


#: ../mail/evolution-mail.schemas.in.h:54
"Level beyond which it should the message should be logged."
yes, still.


and it's still "showed" in some string instead of "shown", don't remember which one.
Comment 1 André Klapper 2008-02-03 02:05:39 UTC
> #: ../mail/evolution-mail.schemas.in.h:107
> "Show the email of the sender in the messages composite column in the message
> list."
> srini, what is a composite column? i have NEVER heard that before.

it took me some time to realize that you meant "email address" in fact, and not "email". i think i understood what it means, but i'm still not sure...
Comment 2 André Klapper 2008-02-03 02:06:47 UTC
(In reply to comment #1)
> > #: ../mail/evolution-mail.schemas.in.h:107
> > "Show the email of the sender in the messages composite column in the message
> > list."
> > srini, what is a composite column? i have NEVER heard that before.
> 
> it took me some time to realize that you meant "email address" in fact, and not
> "email". i think i understood what it means, but i'm still not sure...
> 

btw, isn't that the "From" column in the message list? if so, say so. ;-)
Comment 3 Srinivasa Ragavan 2008-02-04 06:43:25 UTC
Composite column is the new column I made for vertical view (2 row column). Sorry to be so confusing.
Comment 4 Srinivasa Ragavan 2008-02-04 06:43:59 UTC
I would get someone fix it. 
Comment 5 Suman Manjunath 2008-02-05 06:39:24 UTC
Created attachment 104461 [details] [review]
patch for string changes

(In reply to comment #0)
> #: ../addressbook/addressbook.error.xml.h:21
> "The address book {0} is read only, thus you cannot add contacts there. 
> Select other address book, please."
> #: ../calendar/calendar.error.xml.h:64
> "You have a read-only calendar source selected. Change to Calendar View and 
> highlight a calendar that can accept appointments."
> please sync the style of these messages, the first one is not that helpful...

why I chose not to include the addrbook/calendar name while displaying the message: sometimes you'll end up with strings like "The calendar 'Calendar' is a read-only calendar source. ... "

> #: ../plugins/mail-notification/apps-evolution-mail-notification.schemas.in.h:2
> "Blink icon in notification area."
> #: ../plugins/mail-notification/apps-evolution-mail-notification.schemas.in.h:4
> "Enable icon in notification area."
> do i have to choose between the blink icon and the enable icon? or do you
> personally blink the icon for me? :)

these are two different things: the first one enables/disables mail notification by displaying an icon in the notification area. The second one is to choose between a blinking/non-blinking icon to notify. Obviously, this is applicable iff 'mail notification by displaying an icon in the notification area' is enabled.

> #: ../mail/evolution-mail.schemas.in.h:54
> "Level beyond which it should the message should be logged."
> yes, still.

guess it was this bad sometime before. the string in SVN trunk seems okay to me.
Comment 6 André Klapper 2008-02-05 08:07:17 UTC
You have selected a read-only calendar source. Switch to Calendar View and highlight a calendar that can accept appointments.
You have selected a read-only address book source. Switch to Contacts View and highlight an address book that can accept new contacts.

i said "sync" in the sense of... "sync" ;-)


+   Show the email of the sender in a separate column, in the message list.
why that comma? (and i personally still don't get the functionality difference)
and can you PLEASE make "email" "email address"? you don't show the email. you show the email address here!

that blink/enable string and the "showed" issue have been fixed already, or why aren't they part of the patch?
Comment 7 Suman Manjunath 2008-02-05 10:03:36 UTC
Created attachment 104467 [details] [review]
patch for string changes

(In reply to comment #6)
> You have selected a read-only calendar source. Switch to Calendar View and
> highlight a calendar that can accept appointments.
> You have selected a read-only address book source. Switch to Contacts View and
> highlight an address book that can accept new contacts.
> 
> i said "sync" in the sense of... "sync" ;-)

eh? I really don't think those strings could be sanitized anymore. :(

> +   Show the email of the sender in a separate column, in the message list.
> why that comma? (and i personally still don't get the functionality difference)
> and can you PLEASE make "email" "email address"? you don't show the email. you
> show the email address here!

yeah.. i missed that one.. 

> that blink/enable string and the "showed" issue have been fixed already, or why
> aren't they part of the patch?

the blink/enable string has been fixed already. 
the "showed" issue: it was/is a part of the patch :-/ (second hunk in evolution-mail.schemas.in)
Comment 8 André Klapper 2008-02-05 11:49:17 UTC
> (In reply to comment #6)
> > You have selected a read-only calendar source. Switch to Calendar View and
> > highlight a calendar that can accept appointments.
> > You have selected a read-only address book source. Switch to Contacts View and
> > highlight an address book that can accept new contacts.
> > 
> > i said "sync" in the sense of... "sync" ;-)
> 
> eh? I really don't think those strings could be sanitized anymore. :(

:) there's a "new" in one string. in general i would prefer a lot to display the source names instead, but perhaps you're right (and strings with a "{0}" in them are never displayed translated so it's probably good to not show the source name)
Comment 9 Suman Manjunath 2008-02-07 10:27:28 UTC
Created attachment 104625 [details] [review]
patch for string changes

I could not come up with a better string :(

Like Andre mentioned (on IRC), a handy 'Help' button would do more good.
Comment 10 André Klapper 2008-02-07 16:32:21 UTC
looks good. keep in mind that stirng freeze is on monday.
Comment 11 Srinivasa Ragavan 2008-02-08 06:08:30 UTC
Suman, one patch is missing.

To pass the calendar name to "prompt-read-only-cal". With out it it would crash.

e_error_new (NULL, "calendar:prompt-read-only-cal", NULL); is how it is called.

Something like this.. e_error_run (NULL, "addressbook:error-read-only", e_source_peek_name (e_book_get_source (book)), NULL);

should be done for calendar also.

Marking this as commit now but with other patch ;-)
Comment 12 Suman Manjunath 2008-02-10 17:14:17 UTC
thanks for spotting that one srag.. :-)

Patch updated and committed to SVN trunk as r34982
(http://svn.gnome.org/viewvc/evolution?view=revision&revision=34982)

String changes announced to gnome-doc-list and gnome-i18n (dated 10th)
(http://mail.gnome.org/archives/gnome-doc-list/2008-February/date.html
http://mail.gnome.org/archives/gnome-i18n/2008-February/date.html)

If other such issues are being tracked as separate bugs, then this one could be closed :-)
Comment 13 Matthew Barnes 2008-03-11 00:36:35 UTC
Bumping version to a stable release.
Comment 14 Matthew Barnes 2008-08-28 14:15:23 UTC
Sounds like this can be closed?  If not, reopen with a clarification of what issues remain.