Closed Bug 378930 Opened 16 years ago Closed 4 years ago

tree widgets have extra space when CSS pixels scaled (high dpi mode)

Categories

(Core :: XUL, defect)

defect
Not set
normal

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
Assignee: jag → Jan.Varga
Component: XP Toolkit/Widgets → XP Toolkit/Widgets: Trees
QA Contact: xptoolkit.widgets → xptoolkit.trees
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...
Hrm, maybe it's not so bad, then.  I suppose that's there for images that go in trees?
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
(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.
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.
My patch to bug 386640 appears to solve this.
Depends on: 386640
Component: XP Toolkit/Widgets: Trees → XUL
QA Contact: xptoolkit.trees → xptoolkit.widgets
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: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.