Closed Bug 206008 Opened 21 years ago Closed 1 year ago

needs control on the number of unread messages in newsgroups

Categories

(MailNews Core :: Localization, defect)

defect

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: tsahi_75, Unassigned)

Details

Attachments

(2 files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; he-IL; rv:1.3) Gecko/20030312
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; he-IL; rv:1.3) Gecko/20030312

in the mail/news reader, a localizer has no control over the position of the
number of unread messages and the brackets that surroud it next to a newsgroup
name, and as a result, in a RTL localization (hebrew, arabic etc.) the closing
bracket is at the left end instead of the right end.

we should have something like

%group% (%S)

to control the position of the group name and the number of unread messages.

Reproducible: Always

Steps to Reproduce:
1. aligning the interface to the right: add these lines to the file intl.css, in
the locale\en-US\global, in the en-US.jar file (the language pack file, in the
chrome folder):

window,dialog,wizard,page {
  direction: rtl;
}

menu { direction: rtl; }

outliner { direction: rtl; }


2.open Mail and Newsgroups
3.register to a Usenet server, such as news.mozilla.org
4.register to a newsgroup on that server
5.look at the list of group(s) under the usenet account in the accounts pane
(should be on the right side after step 1).


Actual Results:  
i get the following:

(n.p.m.l10n(256

Expected Results:  

n.p.m.l10n(256)

this behaviour is correct according to the BiDi algorithm, but should be
overridden somehow.
Attached image screen shot
-> mailnews l10n
Assignee: mkaply → rchen
Status: UNCONFIRMED → NEW
Component: BiDi Hebrew & Arabic → Localization
Ever confirmed: true
Product: Browser → MailNews
QA Contact: zach → marina
Summary: [RFE] needs control on the number of unread messages in newsgroups → [RFE] needs control on the number of unread messages in newsgroups
actually it's not just news groups, but pop/imap accounts too, where you see the
number of unread messages next to the accunt name.
Currently the string is created by
nsMsgFolderDataSource::CreateUnreadMessagesNameString() at
http://lxr.mozilla.org/seamonkey/source/mailnews/base/src/nsMsgFolderDataSource.cpp#1125

AFAIK there's no reason why the code couldn't be changed to use a localizable
format string, but you would still need some subtle use of Unicode control
characters to solve the problem.
i'll handle it :) just put it in the language pack somewhere.
Comment on attachment 123931 [details] [diff] [review]
Localizable format string

sspitzer, can you review?
Attachment #123931 - Flags: superreview?(sspitzer)
Attachment #123931 - Flags: review?(sspitzer)
Comment on attachment 123931 [details] [diff] [review]
Localizable format string

simon, thanks for working on this.

but if we are going to fix this, we should go all the way.

why not:

folderNameWithUnreadCount=%1$S (%2$S)

that avoids the leading space thing, which can't be localized, right?
Attachment #123931 - Flags: superreview?(sspitzer)
Attachment #123931 - Flags: review?(sspitzer)
Attachment #123931 - Flags: review-
also, you'll need l10n approval for this, ping robinf/bobj for that (unless you
have it already)
i agree with seth. i'm not sure something like "(%S)" will be enough to control
the posistion of the brackets.
what's up with this bug?
Summary: [RFE] needs control on the number of unread messages in newsgroups → needs control on the number of unread messages in newsgroups
Product: MailNews → Core
Assignee: rchen → nobody
OS: Windows XP → All
QA Contact: marina → localization
Hardware: PC → All
Product: Core → MailNews Core
Severity: normal → S3

This appears to be correct now (at least for "he").

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: