Allow to set opacity on ::-moz-tree-cell-text

RESOLVED FIXED in mozilla7

Status

()

Core
Widget
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: mak, Assigned: mak)

Tracking

Trunk
mozilla7
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fixed-in-places])

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

6 years ago
same as bug 664127 but for moz-cell-text. Unfortunately the icon is not enough for cut, since in some view users may hide icons, so I need to be able to set text opacity differently from the icon (the text should be more readable).

While the patch is trivial (same as the other bug, but in PaintText), I'm unable to make a reftest for this, the fact that I set opacity on the tree or on the cell-text makes subpixel-AA act differently and the test fails even if looking at screenshots they are equal.
I've tried using sans-serif or monospace, adding padding or other tricks I've seen in other reftests, but no luck.
(Assignee)

Comment 1

6 years ago
Created attachment 542914 [details] [diff] [review]
patch v1.0

Any idea what I could try to solve the subpixel-AA problem?
I thought the easy way to test this was just to add a label and opacity on cell-test to the test I added in bug 664127, but I have not find a way to bypass that issue.
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Attachment #542914 - Flags: review?(roc)
(Assignee)

Comment 2

6 years ago
Created attachment 542955 [details] [diff] [review]
patch v1.1

Nevermind, I found a way.  Using opacity: 0 I ensure opacity works and at the same time there is no antialiasing risk since nothing appears.
Attachment #542914 - Attachment is obsolete: true
Attachment #542955 - Flags: review?(roc)
Attachment #542914 - Flags: review?(roc)
Comment on attachment 542955 [details] [diff] [review]
patch v1.1

Review of attachment 542955 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #542955 - Flags: review?(roc) → review+
(Assignee)

Comment 4

6 years ago
pushed with other stuff, will merge as soon as I get a green run
http://hg.mozilla.org/projects/places/rev/f76ebe0465e1
Whiteboard: [fixed-in-places]
(Assignee)

Comment 5

6 years ago
http://hg.mozilla.org/mozilla-central/rev/f76ebe0465e1
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
You need to log in before you can comment on or make changes to this bug.