yairEO/tagify

appendTarget hiding/selecting issues for shadowdom

Aperta

#1390 aperta il 17 ott 2024

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)HTML (456 fork)github user discovery
Help wantedSuggestion

Metriche repository

Star
 (3880 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Prerequisites

  • I am running the latest version
  • I checked the documentation and found no answer
  • I checked to make sure that this issue has not already been filed

💥 Demo Page

I can do it in some days, i need some time to build a simple lit element for this.

Explanation

  • What is the expected behavior?

There are a lot of document.body checks in the code which doesn't work in shadowdom. It should be the appendTarget element:

https://github.com/yairEO/tagify/blob/master/src/parts/dropdown.js#L155 https://github.com/yairEO/tagify/blob/master/src/parts/dropdown.js#L218

  • What is happening instead?

Checking in the document.body instead of appendTarget

  • What error message are you getting?

A lot of glitches.

Guida contributor

Direzione di ricerca
Sostituisci i riferimenti a document.body con appendTarget nelle righe 155 e 218 di dropdown.js per risolvere i problemi con Shadow DOM.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptDOM
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70