931463 - Shader editor programs checkboxes misaligned on Linux

Status: RESOLVED FIXED

(DevTools Graveyard :: WebGL Shader Editor, defect)

Comment 1

[Victor Porof [:vporof][:vp]]

Attachment: webgl-checkboxes.patch

Comment 2

[Panos Astithas (he/him) [:past] (please ni?)]

Comment on attachment 822876 [details] [diff] [review]
webgl-checkboxes.patch

Review of attachment 822876 [details] [diff] [review]:
-----------------------------------------------------------------

I would only note that you are converting a number of child selectors to the more expensive descendant selectors. Is this really necessary?

https://developer.mozilla.org/en-US/docs/Web/Guide/CSS/Writing_efficient_CSS?redirectlocale=en-US&redirectslug=CSS%2FWriting_Efficient_CSS#Avoid_the_descendant_selector.21

Comment 3

[Victor Porof [:vporof][:vp]]

(In reply to Panos Astithas [:past] from comment #2)
> Comment on attachment 822876 [details] [diff] [review]
> webgl-checkboxes.patch
> 
> Review of attachment 822876 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I would only note that you are converting a number of child selectors to the
> more expensive descendant selectors. Is this really necessary?
> 

I like that you noticed this :) Those nodes are not direct descendants on Linux anymore. There's a different internal DOM structure for the checkboxes on that magnifique platform. So instead of duplicating the rules in the designated themes/linux/shadereditor.css file, or having two different selectors for the same thing in the themes/common/shadereditor.inc.css file, I resorted to the simpler approach of converting the direct descendant selector to a more generic one.

It's also worth noting that, although the MDN article advises against generic descendant selectors (and I've been using the arguments in that article more than once when cleaning up my own styles *and* convincing others of the validity of efficient selectors), it was brought to my attention many times that our CSS parsers are very efficient now and "writing efficient selectors" is overrated. Moreover, since (in Gecko at least) selectors are parsed from right to left, and there are significantly less checkboxes than other elements in the Shader Editor, worrying about performance is not entirely justified at this time.

Comment 4

[Phil Ringnalda (:philor)]

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