Hammerspoon/hammerspoon

windowHints hotkey modal clashes with user defined modals

Open

#606 aberto em 14 de out. de 2015

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Objective-C (688 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (15.415 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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!

Guia do colaborador

Direção de pesquisa
Examine o módulo windowHints em extensions/hints/init.lua. A variável global 'k' na linha 175 deve ser alterada para uma variável local ou renomeada para evitar conflitos com modais definidos pelo usuário. Verifique se há outras variáveis globais no mesmo módulo que possam causar problemas semelhantes. Também verifique se a alteração não quebra nenhuma outra funcionalidade que dependa da variável global 'k'.
Pilha de tecnologia
None
Domain
desktop
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Lua basicsHammerspoon setup
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80