youtube-challenge-electron-tray
youtube-challenge-electron-tray copied to clipboard
rocketseat-content
Feature/auto launch
Opa @luizjr, beleza? Obrigado pela contribuição. Alguns pontos:
- Criamos um README pro projeto por isso acho que você pode remover o seu, ou no máximo atualizar o existente com o que achar necessário;
- Vi que você criou um menu de settings mas pelo jeito quando clica nele a aplicação fecha, acho que devíamos ter um controle maior sobre e o usuário escolher se o app deve ou não iniciar com o sistema;
- Sua PR tem vários commits que podiam ser unidos em um só, recomendaria talvez você criar uma nova PR apenas com os commits necessários;
Opa @luizjr, beleza? Obrigado pela contribuição. Alguns pontos:
- Criamos um README pro projeto por isso acho que você pode remover o seu, ou no máximo atualizar o existente com o que achar necessário;
- Vi que você criou um menu de settings mas pelo jeito quando clica nele a aplicação fecha, acho que devíamos ter um controle maior sobre e o usuário escolher se o app deve ou não iniciar com o sistema;
- Sua PR tem vários commits que podiam ser unidos em um só, recomendaria talvez você criar uma nova PR apenas com os commits necessários;
Essa seria a função do settings, no momento eu havia apenas habilitado ele no start do sistema, o settings poderia ser uma janela para configurar mais coisas, incluindo se ele inicia ou não com o sistema.
Agora você deletou o README que nós criamos para o projeto hahaha. Seria legal talvez você criar uma nova PR, fechar essa. E nessa nova PR incluir apenas o código do Auto Launch. Mas acho que antes de você enviar a PR seria legal você tentar criar uma janela pro usuário configurar se ele quer ou não que a aplicação abra no startup da máquina ao invés de deixar essa configuração pré-setada.
Agora você deletou o README que nós criamos para o projeto hahaha. Seria legal talvez você criar uma nova PR, fechar essa. E nessa nova PR incluir apenas o código do Auto Launch. Mas acho que antes de você enviar a PR seria legal você tentar criar uma janela pro usuário configurar se ele quer ou não que a aplicação abra no startup da máquina ao invés de deixar essa configuração pré-setada.
O README eu já resolvi. Acredito que você poderia criar essa janela de Settings eu e outros usaríamos. Então eu adicionaria a opção de abrir ou não na inicialização do sistema.
Vou ver se consigo um tempo pra fazer isso, semana OmniStack se aproximando meu tempo fica complicado haha
@diego3g @luizjr Ficou bonitinho acho :)
Acho que é isso que precisa! Vai codificar?
Eu só adicionei uma opção Preferences no menu, que abre essa janela. html e css estão redondos. Agora fazer o treco funcionar de fato to tendo uns problemas pra fazer rodar
@nidib Você já fez um Pull Request? Preciso que o @diego3g mescle seu PR para eu trabalhar nele.
@nidib Você já fez um Pull Request? Preciso que o @diego3g mescle seu PR para eu trabalhar nele.
Dessa feature da screenshot?
@nidib Você já fez um Pull Request? Preciso que o @diego3g mescle seu PR para eu trabalhar nele.
Dessa feature da screenshot?
Sim!!
Nao fiz ainda. Estava esperando o @diego3g dar um merge na #20. Já que o spawn vai mudar de novo. Posso fazer mesmo assim? Da nada? haha
Nao fiz ainda. Estava esperando o @diego3g dar um merge na #20. Já que o spawn vai mudar de novo. Posso fazer mesmo assim? Da nada? haha
@nidib Eu acredito que isso pode lhe ajudar ha fazer o checkbox do auto start funcionar.
var toggleAutoLaunchTemplate = {
label: 'menuAutoStart',
type: 'checkbox',
click: function() {
init.save('shouldAutoLaunch', !init.restore('shouldAutoLaunch'));
init.restore('shouldAutoLaunch') ? CodeTrayAppLauncher.enable() : CodeTrayAppLauncher.disable(); // eslint-disable-line
},
};