twbs/bootstrap

Expounding on "opt-in for performance reasons"

Open

#29.378 aperta il 6 set 2019

Vedi su GitHub
 (7 commenti) (10 reazioni) (0 assegnatari)MDX (78.904 fork)batch import
docshelp wantedjsv5v6

Metriche repository

Star
 (174.240 star)
Metriche merge PR
 (Merge medio 6g 11h) (47 PR mergiate in 30 g)

Descrizione

In the documentation for both Tooltips and Popovers is the caveat: "opt-in for performance reasons, so you must initialize them yourself." But I'm having trouble finding a detailed explanation of "performance reasons." The caveat is at least 6 years old, which makes me wonder if it's still true.

Also, many people miss this caveat when reading the documentation, for example #24985 and #15458. So it would be great if Tooltips and Popovers could actually be automatically initialized in a performant way. That might also pair nicely with using MutationObserver to automatically remove Tooltips in v5.

Guida contributor

Direzione di ricerca
Esamina il blame collegato (https://github.com/twbs/bootstrap/blame/c6aae570b1bc608f5ca4b9ce9085ba9929edeb13/docs/javascript/tooltips.md#L37 L38) per comprendere il contesto originale delle prestazioni. Rivedi le issue #24985, #15458 e #28964 per vedere la confusione comune degli utenti e le discussioni pertinenti. Proponi una documentazione aggiornata che spieghi chiaramente i compromessi in termini di prestazioni e suggerisci eventualmente l'inizializzazione automatica utilizzando MutationObserver come discusso in #28964.
Tech stack
javascriptcsshtml
Dominio
frontend
Tipo issue
Documentazione
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
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65