Closed
Bug 335739
Opened 19 years ago
Closed 19 years ago
[suiterunner] File and Edit menu have no labels and have missing entries
Categories
(SeaMonkey :: UI Design, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mcsmurf, Assigned: kairo)
References
Details
Attachments
(1 file)
(deleted),
patch
|
neil
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
I noticed in current suiterunner builds under Windows the labels in the menubar are missing (just blank space) and also in the menus itself some labels of menu entries are missing (like the Work Offline label, Undo, Cut, Copy, Paste and some others).
Assignee | ||
Comment 1•19 years ago
|
||
This has been caused by Axel's checkin for bug 328956, as he unveiled that we were still accessing the releaseURL entity from global-region/region.dtd which he correctly removed from toolkit (see the chrome error at the bottom of the screen).
This should live in suite-specific code of course, either communicator-region or branding.
Blocks: suiterunner, 334478
Depends on: 328956
Assignee | ||
Comment 2•19 years ago
|
||
I chose to move it to branding as we already have a .dtd there, while communicator-region only has a .properties and both would be correct to about the same extent IMO.
The Makefile needs to forward the app version var though so that we can use it in the URL.
Assignee: jag → kairo
Status: NEW → ASSIGNED
Attachment #220084 -
Flags: superreview?(neil)
Attachment #220084 -
Flags: review?(neil)
Comment 3•19 years ago
|
||
Comment on attachment 220084 [details] [diff] [review]
move releaseURL to branding for suiterunner
>\ No newline at end of file
>\ No newline at end of file
r+sr=me with these fixed. The first one was particularly silly, as your edits weren't at the end of the file :-P
Attachment #220084 -
Flags: superreview?(neil)
Attachment #220084 -
Flags: superreview+
Attachment #220084 -
Flags: review?(neil)
Attachment #220084 -
Flags: review+
Assignee | ||
Comment 4•19 years ago
|
||
Checked in.
BTW, for any developer listening in this bug: NEVER ever add ifdefs to locale files!
That said, this one is temporarily needed so that brand.dtd and region.dtd don't clash in xpfe-based suite - but plase don't follow that example in other places.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•