Closed
Bug 99729
Opened 23 years ago
Closed 23 years ago
Remove uses of halign="left"
Categories
(Core Graveyard :: Security: UI, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
Future
People
(Reporter: stdowa+bugzilla, Assigned: rangansen)
Details
Attachments
(1 file)
|
3.67 KB,
patch
|
Details | Diff | Splinter Review |
halign="left" is unnecessary on <hbox>'s, it should be removed from the 3 psm pref files that currently use it.
|
Reporter | |
Comment 1•23 years ago
|
||
|
|
Reporter | |
Updated•23 years ago
|
|
||
Comment 2•23 years ago
|
||
->rangansen Rangan. Follow up with reviewers, and check in onto the trunk.
Assignee: ssaux → rangansen
Priority: -- → P2
Target Milestone: --- → Future
|
Assignee | |
Comment 3•23 years ago
|
||
This patch seems to work fine on NT - apparently, having the 'halign=left' staff is really redundant. But, can removing this cause any impact on other platforms? I could not find 'halign' on any of the available xul documentations, though it seems that the implementations actually supports it. Otherwise, r=rangansen
|
|
Reporter | |
Comment 4•23 years ago
|
||
I don't think removing this will cause any issues on other platforms, because I think <hbox>'s default align value is left. cc'ing hewitt for input.
|
||
Updated•23 years ago
|
QA Contact: bsharma → junruh
|
||
Comment 5•23 years ago
|
||
There is no platform-specific behavior to worry about here.
|
|
||
Comment 6•23 years ago
|
||
sr=hewitt
|
|
Assignee | |
Comment 7•23 years ago
|
||
Patch checked in ...
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
|
||
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•