17621 - Leave toolbar button separators in XUL

Status: VERIFIED WONTFIX

(SeaMonkey :: General, defect, P3)

Description

[Michael Lowe]

It seems that you are removing the toolbar button separators for the new skin.
I think it is better they are left in the XUL code because in some skins people
will want them displayed.   A better approach if you don't want them displayed
with this skin is to hide them with the appropriate CSS rules.

Comment 1

[german]

XUL springs are the official (more appropriate and generalizeable) method of
achieving separators in XUL. Properly named and ID'd springs can then display
whatever style they want (such as borders, ridges etc).

Comment 2

[Claudius Gayle]

marking VERIFIED

Comment 3

[Michael Lowe]

If I look at the file editorOverlay.xul, I don't see any XUL springs either.
Are they located for the editor somewhere else?

Comment 4

[german]

No XUL springs can be anything you want to be they are a tag that's already an
official part of XUL. The exact specification of the look, spacing etc. is part
of a skin.
The older spearator was specified as html:div which is not as flexible and has
other problems when used inside boxes.
In other words if you want spacers in your skin you simply specify the springs
in your XUL file, then use a class (for multiple occurences) or id (for single
ocurrences) and refer to that in your style sheet. As I said the particular
design should really not be hard-coded into a global XUL.css but instead is part
of a specific skin design.

Comment 5

[leger]

Moving all UE/UI bugs to new component: User Interface: Design Feedback
UE/UI component will be deleted.