UI Tests Results

    1 files     98 suites   38s ⏱️
1 031 tests 1 031 ✅ 0 💤 0 ❌
1 050 runs  1 050 ✅ 0 💤 0 ❌

Results for commit 77f406f.

♻️ This comment has been updated with latest results.

Thanks for the contribution. But, we can't accept this PR due to the reasons I've listed in the issue, which I also added below.

Since it's in the slot, the user is in total control. So, the user can do:

<q-knob
  ...

  <q-avatar size="60px">
    <img draggable="false" src="https://cdn.quasar.dev/logo-v2/svg/logo.svg">
  </q-avatar>
</q-knob>

or

<q-knob
  ...

  <q-avatar size="60px">
    <img class="no-pointer-events" src="https://cdn.quasar.dev/logo-v2/svg/logo.svg">
  </q-avatar>
</q-knob>

Using images is not a really frequent use case, and the solution is not anywhere near being hard, solving this without causing any potential problems to other use cases or preserving the performance is not guaranteed.

So, the proper solution for this would be to document this case in the docs, and possibly integrate it into the example you pointed out as well. For that reason, I'm afraid I will have to deny your PR. But, I'll keep this issue until it gets documented.

Originally posted by @yusufkandemir in #17812