possible CSS mistake - ``#pql`` and `` #rgbwrap`` div not hidden by default

[softhack007]

WLED/wled00/data/index.css

Lines 691 to 696 in b60313e

	/* wrapper divs hidden by default */
	#liveview, #liveview2D, #roverstar, #pql
	#rgbwrap, #swrap, #hwrap, #kwrap, #wwrap, #wbal, #qcs-w, #hexw,
	.clear-icon, .edit-icon, .ptxt {
	display: none;
	}

I'm not at expert in CSS, but it could be that a comma is missing at the end of line 692?

Asked the rabbit and he explains is like this

When there is no comma between #pql and #rgbwrap, the parser treats that gap as a descendant combinator. The selector list effectively contains a compound selector "#pql #rgbwrap" (rgbwrap inside pql) instead of two separate selectors "#pql" and "#rgbwrap". That means #pql itself will NOT be matched/hidden unless #rgbwrap is a descendant of it.

[DedeHai]

good find, the bug should not have any consequence though: pql is the preset quickload, which is empty at page load and only filled by JS. rgbwrap are the rgb sliders and get enabled/hidden dynamically in JS, they just might show up briefly during page load, so not a big deal.

[blazoncek]

Not true. It prohibits #rgbwrap of being properly rendered.

[DedeHai]

in a special case? seems to render fine?

[blazoncek]

See rodent's explanation.

[github-actions]

Hey! This issue has been open for quite some time without any new comments now. It will be closed automatically in a week if no further activity occurs.
Thank you for using WLED! ✨