Ceci est principalement destiné aux mainteneurs de Vimari (que je crois être @danielcompton et @simeg)
J'utilise Vimari depuis un certain temps maintenant, mais il manque certaines fonctionnalités que moi et d'autres aimerions voir. Ceux-ci inclus:
- Personnalisation (#171, #166, #155, #153, #142, #98)
- Meilleure isolation de mode (
insertvsnormal) (#170, #168, #148, #110, #102) - Défilement fluide (#157, #83)
- Une page d'aide (#158)
Certaines de ces fonctionnalités sont disponibles depuis un certain temps dans les relations publiques du projet, que j'ai combinées dans ma propre version de Vimari sur mon propre fork .
Mon fork comprend actuellement :
- mettre à jour la bibliothèque de pièges à souris
- support de configuration (principalement par le travail de @nieldm),
- défilement fluide
- mode isolé
normaltel que des problèmes comme #168 ne se produisent pas
Avant de pouvoir fusionner ces modifications dans le dépôt principal, j'aimerais :
- mettre en place un indicateur indiquant quand vous êtes en mode insertion.
- introduire une sorte de superposition avec une page d'aide (bien que cela puisse être fait séparément)
- nettoyage du code hérité lié au fonctionnement de l'extension avant Safari 12. Il semble y avoir du code inutilisé qui devrait probablement être supprimé.
- en vous assurant que l'extension fonctionne correctement sur macOS Big Sur.
Je me demande s'il y a un intérêt à ramener une partie de cette fonctionnalité dans ce référentiel principal et à l'envoyer à tous ceux qui utilisent Vimari depuis l'App Store. Comme je consacre déjà une partie de mon temps libre à mon propre fork, je serais prêt à consacrer du temps supplémentaire pour ramener ces fonctionnalités au référentiel principal. Cependant, comme le repo est assez silencieux depuis un certain temps maintenant, j'aimerais en discuter avant de commencer le travail.
J'espère que c'est le bon endroit pour en discuter, s'il vous plaît laissez-moi savoir s'il y a un intérêt à cela et comment nous pourrions nous y prendre.
nbelzer
Tous les 10 commentaires
Bonjour.
Je ne maintiens plus ce dépôt, mais ça me semble génial !
Ma recommandation serait de ne pas viser une intégration big bang mais de faire plusieurs petits PR descriptifs. Votre code peut avoir un sens pour vous, mais pour les responsables, il peut être difficile de comprendre les changements de code s'ils sont regroupés et manquent de bons messages de validation. Ce n'est probablement pas nouveau pour personne mais j'ai l'impression que cela vaut la peine d'être répété 🙂
simeg
le 12 juil. 2020
Salut @nbelzer , merci d'avoir fait tout ce travail. Comme vous pouvez probablement le constater, je n'ai pas eu beaucoup de temps pour Vimari au cours des six derniers mois à cause de COVID + d'autres trucs personnels. Si vous le souhaitez, je suis heureux de vous inviter en tant que responsable du dépôt.
À plus long terme, j'ai été ravi de voir que Safari va prendre en charge WebExtensions . En théorie, cela nous permettrait d'unifier la base de code avec Vimium. Je vois que vous êtes déjà impliqué dans la discussion sur https://github.com/philc/vimium/issues/3610.
danielcompton
le 14 juil. 2020
Merci pour votre réponse @danielcompton , j'espère que vous allez bien !
Je serais heureux d'aider en tant que mainteneur. Maintenant que j'ai un peu de temps libre pour (au moins) les 2 prochains mois, je pense que je pourrais être utile pour introduire les fonctionnalités mentionnées ci-dessus dans Vimari. Je dois mentionner que je suis assez nouveau sur la scène open source, donc un œil vigilant serait bon d'avoir.
nbelzer
le 14 juil. 2020
@nbelzer Merci pour l'extension mise à jour.
J'ai installé Vimari depuis l'App Store, remplacé l'application dans Applications depuis votre fork et l'ai activée dans les préférences de Safari.
Cependant, il n'y a pas de bouton pour ouvrir le fichier de configuration, userSettings.json.
Un indice sur l'emplacement de ce fichier ?
srikat
le 16 juil. 2020
Hé @srikat , merci de l'avoir essayé. Pour obtenir ces fonctionnalités, vous devez compiler et installer l'application à partir de mon fork sur la branche feature/access_configuration . Si tout est réussi, vous pouvez ouvrir le panneau Vimari (en cliquant sur l'icône d'extension dans Safari) et appuyer sur "Ouvrir le fichier de configuration". Mais sachez qu'il s'agit bien d'une branche expérimentale qui n'était au départ que pour mon usage personnel. Comme mentionné ci-dessus dans ce numéro, je vais travailler sur l'intégration de ces fonctionnalités (comme la configuration) dans le dépôt principal ici.
nbelzer
le 17 juil. 2020
Merci. Je vais attendre qu'il soit en binaire.
srikat
le 17 juil. 2020
@nbelzer Merci pour votre excellent travail, je me demande juste quand prévoyez-vous de fusionner vos nouvelles fonctionnalités avec le référentiel principal ? J'ai hâte de l'essayer. ^.^
DarkAlexWang
le 30 juil. 2020
@DarkAlexWang c'est agréable à entendre, j'espère bientôt. L'oeuvre est déjà disponible dans les pr #175 et #176 comme vous avez pu le voir. Cependant je préfère que Daniel Compton y jette un œil avant de fusionner.
nbelzer
le 30 juil. 2020
@nbelzer Compris , attendra la fusion. Merci mec.
DarkAlexWang
le 30 juil. 2020
À l'exception de la page d'aide, les fonctionnalités de mon fork ont toutes été implémentées dans la branche master. Ils sont déjà disponibles via la dernière version bêta pour les personnes intéressées.
nbelzer
le 22 août 2020
Questions connexes
danielcompton
·
19Commentaires
mihar
·
3Commentaires
jonchoi
·
7Commentaires
dragonma-backup
·
10Commentaires
Hultner
·
9Commentaires
Commentaire le plus utile
Salut @nbelzer , merci d'avoir fait tout ce travail. Comme vous pouvez probablement le constater, je n'ai pas eu beaucoup de temps pour Vimari au cours des six derniers mois à cause de COVID + d'autres trucs personnels. Si vous le souhaitez, je suis heureux de vous inviter en tant que responsable du dépôt.
À plus long terme, j'ai été ravi de voir que Safari va prendre en charge WebExtensions . En théorie, cela nous permettrait d'unifier la base de code avec Vimium. Je vois que vous êtes déjà impliqué dans la discussion sur https://github.com/philc/vimium/issues/3610.