425916 - Improve look of mac treerows when in editing mode

Status: RESOLVED FIXED

(Toolkit :: Themes, defect)

Description

[Stefan [:stefanh]]

Spin-off from bug 420726, comment #12.

In summary (pinstripe/global/tree.css):

391 /* ::::: editable tree ::::: */
 392 
 393 treechildren::-moz-tree-row(selected, editing) {
 394   background-color: transparent;
 395 }
 396 
 397 treechildren::-moz-tree-cell-text(selected, editing) {
 398   color: inherit;
 399 }

Those rules doesn't work on odd rows, because of tree:not([hidecolumnpicker="true"]) treechildren::-moz-tree-row(odd) and treechildren[alternatingbackground="true"]::-moz-tree-row(odd) background color rules.

However, we might want to consider styling this so it'll match what Safari does in the bookmarks view (editable list rows, just click-hold on a bookmark). enn, does this sounds reasonable to you? I don't know a lot of editable trees, so I might have missed something here :-)

Comment 1

[Stefan [:stefanh]]

Attachment: v1.0

So, I think what's left here is to fix the actual editing mode. Right now, the highlight color makes it look quite funny in the editbookmark panel when you have changed the selection color in system prefs.

Apple's rows are not as hight as ours, so compared to Safari (edit a bookmark title) we have more space above the text. The space is needed to extend the editable field to the top of the row. When playing with this, I used the editable tree example at https://wiki.mozilla.org/XUL:Tree with dom.allow_XUL_XBL_for_file set to true.

Markus, I'm not sure Dao has a mac. Would you mind giving some feedback?

Comment 2

[Stefan [:stefanh]]

Attachment: before-after

Hmm, did it became better? OK, we're using the correct color. But I dunno about the positioning (margin/padding).

Comment 3

[Stefan [:stefanh]]

Comment on attachment 584286 [details] [diff] [review]
v1.0

Hm, I think I'll try to see if I can make it better...

Comment 4

[Stefan [:stefanh]]

Attachment: XUL test file

Here's a test file to be used locally (set 'dom.allow_XUL_XBL_for_file' to true).

Comment 5

[Stefan [:stefanh]]

Attachment: V1.1

Markus, maybe you could take a look at this? The treechildren rules should not be needed (and we don't want the text color to inherit since we only want the black text color in the input box - see the testcase).

Comment 6

[Markus Stange [:mstange]]

Comment on attachment 588686 [details] [diff] [review]
V1.1

Looks good to me. But Dão should have a look, too.

Comment 7

[Dão Gottwald [:dao]]

Comment on attachment 588686 [details] [diff] [review]
V1.1

screenshot?

Comment 8

[Stefan [:stefanh]]

Attachment: Screenshots

Comment 9

[Stefan [:stefanh]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/e57b01411394

Comment 10

[Joe Drew (not getting mail)]

https://hg.mozilla.org/mozilla-central/rev/e57b01411394