Closed
Bug 378930
Opened 18 years ago
Closed 6 years ago
tree widgets have extra space when CSS pixels scaled (high dpi mode)
Categories
(Core :: XUL, defect)
Core
XUL
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: dbaron, Assigned: janv)
References
Details
In high dpi mode, when a CSS pixel is more than one physical pixel, tree widgets containing text in system fonts have the rows double the height they should be.
Steps to reproduce:
1. start firefox on a fresh profile
2. go to about:config and set layout.css.dpi to 192
3. look at about:config
Actual results: Each row in the tree has the text in a system font (at the system font's size, which is correct), but there's a lot of extra whitespace around the text as though the font had been doubled in size.
Expected results: the tree widget should appear without extra space
Reporter | ||
Updated•18 years ago
|
Assignee: jag → Jan.Varga
Component: XP Toolkit/Widgets → XP Toolkit/Widgets: Trees
QA Contact: xptoolkit.widgets → xptoolkit.trees
Comment 1•18 years ago
|
||
Quote from chrome://global/skin/tree.css:
treechildren::-moz-tree-row {
border: 1px solid transparent;
min-height: 18px;
height: 1.3em;
}
That seems likely to cause issues...
Reporter | ||
Comment 2•18 years ago
|
||
Hrm, maybe it's not so bad, then. I suppose that's there for images that go in trees?
Assignee | ||
Comment 3•18 years ago
|
||
Tree rows used to have fixed height of 18px and they didn't scale with size of fonts. I fixed it by adding min-height and height: 1.3em
Comment 4•18 years ago
|
||
(In reply to comment #3)
> Tree rows used to have fixed height of 18px and they didn't scale with size of
> fonts. I fixed it by adding min-height and height: 1.3em
What exactly do you mean by "used to"? Is there some patch somewhere I'm missing? Or some non-default theme? All 5 instances of tree.css in the tree contain min-height: 18px.
Assignee | ||
Comment 5•18 years ago
|
||
I fixed it very long ago (several years). There must be a patch somewhere in bugzilla. The bug was assigned to me.
I think the tree widget code, method GetRowHeight() should be adjusted to count with scaled CSS pixels.
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: xptoolkit.trees → xptoolkit.widgets
Assignee | ||
Comment 7•6 years ago
|
||
I don't think anyone is going to work on this given the plan to remove the XUL "tree" widget (bug 1446335).
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•