Hammerspoon/hammerspoon

windowHints hotkey modal clashes with user defined modals

Open

#606 ouverte le 14 oct. 2015

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Objective-C (688 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (15 415 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

It seems like windowHints implementation is based on modal hotkeys, and it defines a global variable named "k" for the modal. That global variable was clashing with my own modal variable (also named "k") causing weird behavior. Changing my modal variable name fixed it.

I'm not experienced enough with hammerspoon or lua to fix this myself. Is it possible to use a local variable in the windowHints module? Maybe the windowHints variable should have a less common name?

I believe this is the line causing trouble.

Thanks!

Guide contributeur

Direction de recherche
Examinez le module windowHints dans extensions/hints/init.lua. La variable globale 'k' à la ligne 175 doit être changée en variable locale ou renommée pour éviter les conflits avec les modaux définis par l'utilisateur. Vérifiez s'il existe d'autres variables globales dans le même module qui pourraient causer des problèmes similaires. Vérifiez également que le changement ne casse aucune autre fonctionnalité dépendant de la variable globale 'k'.
Stack technique
Aucun
Domaine
desktop
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Lua basicsHammerspoon setup
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80