Hammerspoon/hammerspoon

windowHints hotkey modal clashes with user defined modals

Open

#606 geöffnet am 14. Okt. 2015

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Objective-C (688 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (15.415 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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!

Contributor Guide

Research-Richtung
Untersuche das windowHints Modul in extensions/hints/init.lua. Die globale Variable 'k' in Zeile 175 sollte in eine lokale Variable geändert oder umbenannt werden, um Konflikte mit benutzerdefinierten Modals zu vermeiden. Prüfe, ob es andere globale Variablen im selben Modul gibt, die ähnliche Probleme verursachen könnten. Stelle außerdem sicher, dass die Änderung keine andere Funktionalität beeinträchtigt, die von der globalen Variable 'k' abhängt.
Tech Stack
Keine
Domain
desktop
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Lua basicsHammerspoon setup
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80