On Mon, Jan 13, 2025 at 7:38 PM Keith Cirkel <chrom...@keithcirkel.co.uk>
wrote:

> > I don't see that the spec says anything about querySelector, but its
> evaluation depends on slot assignments. Should dirty slot assignments be
> updated before running querySelector?
>
> Thanks Rune, I think you're right, we should definitely fix this so it is
> not observable.
>

FWIW, the slot assignments are made up-to-date for querySelector with
:dir() and auto directionality (tested with Chrome and Firefox), but I
cannot find anything in the spec calling this out specifically:

<!DOCTYPE html>
<div id="host">
  <template shadowrootmode="open">
    <style>
      slot:dir(rtl) { background-color: green; }
    </style>
    <slot dir="auto"></slot>
  </template>
  <div id="auto_div" slot="no">&rlm;</div>
</div>
<script>
  const root = host.shadowRoot;
  const slot = root.querySelector("slot");
  console.log(root.querySelector("slot:dir(rtl)"));
  console.log(getComputedStyle(slot).backgroundColor);
  auto_div.removeAttribute("slot");
  console.log(root.querySelector("slot:dir(rtl)"));

console.log(getComputedStyle(host.shadowRoot.querySelector("slot")).backgroundColor);
</script>


> > Looking at wpt.fyi all tests are tentative, so they need to be made
> non-tentative with proper spec references if they match the spec.
>
> Thanks for mentioning this, I've got a PR for this:
> https://github.com/web-platform-tests/wpt/pull/50056
>

Remember to also update the rel=help links to actual spec references.

-- 
Rune Lillesveen

-- 
You received this message because you are subscribed to the Google Groups 
"blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to blink-dev+unsubscr...@chromium.org.
To view this discussion visit 
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CACuPfeS1QunKZk%2Bv_Vxix9eFO-qe_xkMOMzt_jHZA20x0LFPMA%40mail.gmail.com.

Reply via email to