Closed
Bug 99685
Opened 23 years ago
Closed 23 years ago
Need to externalize the initial size of Offline and Diskspace window on account manager
Categories
(SeaMonkey :: MailNews: Backend, defect)
SeaMonkey
MailNews: Backend
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: ji, Assigned: yinglinxia)
Details
(Keywords: l12y)
Attachments
(3 files)
(deleted),
image/jpeg
|
Details | |
(deleted),
image/jpeg
|
Details | |
(deleted),
patch
|
ssu0262
:
review+
sspitzer
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
With the current build, the initial size of Offline and Diskspace window on mail
account manager is too small for Japanese build. The strings and Select button
on that window are truncated. Although the window can be manually resized by
users, it will give users bad first impression of the product. The initial size
of the window needs to be externalized to the resource file, so localization
engineers can adjust the initial size of that window in language packs. I'll
attach a screen shot of Japanese build to illustrate the problem.
A good example is the preferences dialog which has the externalized initial size
valuable in the dtd file.
I'd say its a dup of bug 84692. To much stuff on the panel and things being cut
off with current default size.
Please note that we are requiring the default size be externalized to a resource
file, since the default sizes for localized builds could be different with the
one for English build. If this bug is resolved as a dup of bug 84692, please
make sure the default size goes to a resource file when a fix for bug 84692 is
checked in.
>Please note that we are requiring the default size be externalized to a
>resource file, since the default sizes for localized builds could be different
>with the one for English build.
Wouldn't you want this to apply to All panels then instead of just this one? In
case any panel grows too large?
Yes, we'd like to have this applied to all the panels, and we are more concerned
about if we can adjust the default size w/o modifying the code in localized builds.
Comment 9•23 years ago
|
||
ji - can you check this one to see if this is still true in 6.2?
Reporter | ||
Comment 10•23 years ago
|
||
The problem still exists in 6.2.
Updated•23 years ago
|
Assignee | ||
Comment 13•23 years ago
|
||
I will take a look and try to fix it now.
Assignee | ||
Comment 14•23 years ago
|
||
Comment 15•23 years ago
|
||
Discussed in 2/28/02 Mail & News bug meeting. Decisions was to minus this bug,
because it's not a showstopper for the release, but we will accept the patch if
it gets in in time. Reassigining to yxia
Assignee | ||
Comment 16•23 years ago
|
||
ssu, would you please review the patch, this kind of bug is critical for our
l10n. Thanks.
Comment 17•23 years ago
|
||
Comment on attachment 71777 [details] [diff] [review]
Patch to move the size value of account manager dialog into dtd.
r=ssu
Attachment #71777 -
Flags: review+
Assignee | ||
Comment 18•23 years ago
|
||
mscott and bienvenu, we need a sr now. thanks.
OS: Windows 2000 → All
Hardware: PC → All
Comment 19•23 years ago
|
||
did you have to do something similar for other dialogs, like the pref window?
I'm curious to know what ben / blake think about this.
I'm ok with this, especially given the comments about how for US builds, we
should leave the value alone, but for other languages, this is the only way
(short of picking a smaller font?) to fix this.
Comment 20•23 years ago
|
||
Comment on attachment 71777 [details] [diff] [review]
Patch to move the size value of account manager dialog into dtd.
blake says:
"yeah, that's what prefs does too. it's not a real scalable solution but i
guess it's ok for now."
It forces the localizers to pick a better size, if the default (english) size
is too small.
sr=sspitzer
Attachment #71777 -
Flags: superreview+
Assignee | ||
Comment 21•23 years ago
|
||
Thanks a lot, everybody, it benefits our l10n a lot.
rchen, do you mind to check it in now? I don't have the checkin account yet;-)
Comment 22•23 years ago
|
||
ssu, can you help ying to checkin?
Comment 23•23 years ago
|
||
Comment on attachment 71777 [details] [diff] [review]
Patch to move the size value of account manager dialog into dtd.
a=asa (on behalf of drivers) for checkin to 0.9.9 and the Mozilla trunk.
Attachment #71777 -
Flags: approval+
Comment 24•23 years ago
|
||
patch checked into the 0.9.9 branch and the trunk.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Comment 25•23 years ago
|
||
Using US Commercial trunk builds:
2002030403 on NT 4.0
2002030406 on linux 2.2
2002030412 on Mac 9.2.2
2002030403 on MAC 10.1.3
The offline and disk space panel for mail and news looks the same
for US builds. Tried both Classic and Modern theme. The
fix didn't help the offline & diskspace panel bugs (bug 96541
or bug 84692) but then i don't think the fix was designed to
fix US builds.
Does someone on International QA team want to verify that the
International Builds are fixed? Then mark it as verified?
Or if someone wants to show me, I can try international builds.
Leaving as Resolved until International builds are verified.
Assignee | ||
Comment 26•23 years ago
|
||
This bug fix does nothing about US build, but just put the dialog layout
numbers into the language pack, so the localizer can easily change the dialog
size to fit their language.
If you see anything wrong with the US build, file bugs against US build please.
Assignee | ||
Comment 27•23 years ago
|
||
Tested 03-04 US build, now we can change this dialog size in the language pack.
You can verify this bug now.
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•