Melhora o funcionamento no Windows e ajeita algumas coisas #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
-Variável tray é declarada como null e fora de "app.on('ready')" inicialmente para evitar o garbage collection (o que fazia o tray sumir).
-Render agora usa a variável tray declarada anteriormente fora do escopo da função
-Caso esteja rodando no Windows, o app usa a função exec ao invés de spawn (por algum motivo, spawn não estava funcionando)
Com esses fixes já é possível fazer uma build funcional para Windows (espero hehe, no meu funcionou tranquilo)
Quando há uma lista longa de projetos, o tray é fechado automaticamente quando um é excluído, isso fica incoveniente caso o usuário tenha diversos projetos. O Electron tem um bug conhecido que não deixa o menu/tray ser atualizado dinamicamente, tendo que ser recriado. Entretanto, quando é recriado, o menu/tray fecha automaticamente. A solução foi pegar as coordenadas do tray (pelo mouse) quando o usuário abre ele, e reabri-lo na mesma posição quando o projeto é excluído. Isso gera um blink no tray, mas é mais conveniente do que ter que ficar abrindo toda hora.
WilianZilv
reviewed
Jul 24, 2019
|
@WilianZilv sim! Como faz pra ajeitar aqui no pull-request? Kkkk |
|
Agora não sei, não manjo como funciona o git muito bem ainda, essa foi a minha primeira contribuição pra um projeto huahsush |
|
Acho que é só atualizar a tua master que o pull request atualiza sozinho |
|
Foi! @WilianZilv <3 |
|
Agora é esperar o diego notar que tem gente melhorando o projeto kkk |
|
Marca ele que é sucesso kkkk |
diego3g
approved these changes
Jul 26, 2019
|
Show @renandecarlo, vou testar o projeto no meu Windows em casa só pra garantir e daí aceito o PR :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adiciona um ícone grayscale pro Windows. O antigo ficava preto, grande e feio na taskbar.
O botão esquerdo não estava funcionando pra abrir o tray no Windows.
Conserta o Windows pra abrir com o
child_process.spawnao invés doexec, que o @WilianZilv tinha colocadoRemove o JSON parsing da Store. A Store já salva e interpreta o JSON automaticamente.
Quando remove um projeto da lista, deixa o Tray aberto para poder remover outros sem ter que abrir novamente. Útil pra quando tiver muitos projetos.
O Electron tem um bug que não deixa atualizar/remover o menu dinamicamente. Precisa recriar o menu. Só que quando recria, ele fecha sozinho.
Minha solução foi salvar a posição do mouse quando o usuário abre o app no tray, e aí reabrir manualmente nesta mesma posição quando remove o projeto. O tray da uma piscada, mas é melhor do que ter que ficar abrindo toda hora.
Being able to update a menu item currently showing electron/electron#4685
Update Menu Items electron/electron#937
Support dynamic menu items in electron eclipse-theia/theia#446
Fix #2
O pull é em cima no do @WilianZilv que ainda está aberto.