Add a `dispose()` method to `Docsify` to clean up the instance
Les mainteneurs répondent en général sous 8 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- javascript
- Domaine
- frontend
Piste de recherche
Commencez par localiser la classe Docsify ainsi que la configuration des listeners de changement de route et des gestionnaires de clic sur les anchor-links. Suivez chaque ressource d’instance qui doit être nettoyée, puis vérifiez que l’appel à dispose() empêche l’instance de continuer à répondre ou à effectuer le rendu lorsqu’elle n’est plus nécessaire.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Feature request
Add a dispose() method so that Docsify can be cleaned up when no longer needed.
What problem does this feature solve?
This is needed for embedding Docsify (https://github.com/docsifyjs/docsify/issues/799) on sites that switch to and from a view that has a Docsify instance (maybe they have multiple Docsify instances on different pages).
People currently can not clean up the instance when they will no longer render it.
What does the proposed API look like?
// later
window.$docsify.dispose() // or similar
and for people who will use the Docsify class directly,
const docsify = new Docsify(...)
// later
docsify.dispose()
How should this be implemented in your opinion?
Add a dispose method that cleans up route change listeners, anchor link click handlers, etc.
- Langage dominant
- JavaScript
- Étoiles
- 31.5k
- Forks
- 5.8k
- Merge moyen
- 7 j 18 h
- PR mergées (30 j)
- 9
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de docsifyjs/docsify
-
Query parameter values are truncated at a second equals signPeut-être pris @xu91102 l’a pris il y a 23 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 8 jours
-
Bullet points beginning with inline math: subsequent text formatting not recognised correctlyOuvertebug Triage
Difficulté 3/5 1-2 jours Accessibilité débutants 66/100
Les mainteneurs répondent en général sous 8 jours
-
Cached AJAX responses lose status metadata exposed through vm.route.responsePeut-être pris @xu91102 l’a pris il y a 23 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 8 jours
-
In the Edge browser, the character `{docsify-updated}` is not replacedPeut-être à nouveau libre @AmanUllah687 l’a pris il y a 76 jours, et aucune pull request n’est ouverte. Ouvertewait for information
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
docsifyjs/docsify#2681 · 11 commentaires ·
Les mainteneurs répondent en général sous 8 jours
-
Nested/Recursive includesOuvertefeature request Triage
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
docsifyjs/docsify#2643 · 5 commentaires ·
Les mainteneurs répondent en général sous 8 jours
Toutes les issues de docsifyjs/docsify
Issues similaires
-
[APP/MODULE]Ouverteenhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
awesome-android-root/awesome-android-root#195 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
subhranshudash13-dotcom/Contribo#74 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
qq15725/modern-screenshot#174 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 67/100
ThatGuySam/doesitarm#1114 ·