Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/auto launch #16

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

luizjr
Copy link

@luizjr luizjr commented Jul 30, 2019

Adicionando auto launch

@diego3g
Copy link
Member

diego3g commented Jul 30, 2019

Opa @luizjr, beleza? Obrigado pela contribuição. Alguns pontos:

  1. 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;
  2. 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;
  3. 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;

@luizjr
Copy link
Author

luizjr commented Jul 30, 2019

Opa @luizjr, beleza? Obrigado pela contribuição. Alguns pontos:

  1. 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;
  2. 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;
  3. 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.

@diego3g
Copy link
Member

diego3g commented Jul 30, 2019

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.

@luizjr
Copy link
Author

luizjr commented Jul 30, 2019

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.

@diego3g
Copy link
Member

diego3g commented Jul 30, 2019

Vou ver se consigo um tempo pra fazer isso, semana OmniStack se aproximando meu tempo fica complicado haha

@nidib
Copy link

nidib commented Jul 31, 2019

image

@diego3g @luizjr Ficou bonitinho acho :)

@luizjr
Copy link
Author

luizjr commented Jul 31, 2019

image

@diego3g @luizjr Ficou bonitinho acho :)

Acho que é isso que precisa!
Vai codificar?

@nidib
Copy link

nidib commented Jul 31, 2019

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

@luizjr
Copy link
Author

luizjr commented Jul 31, 2019

@nidib Você já fez um Pull Request?
Preciso que o @diego3g mescle seu PR para eu trabalhar nele.

@nidib
Copy link

nidib commented Jul 31, 2019

@nidib Você já fez um Pull Request?
Preciso que o @diego3g mescle seu PR para eu trabalhar nele.

Dessa feature da screenshot?

@luizjr
Copy link
Author

luizjr commented Jul 31, 2019

@nidib Você já fez um Pull Request?
Preciso que o @diego3g mescle seu PR para eu trabalhar nele.

Dessa feature da screenshot?

Sim!!

@nidib
Copy link

nidib commented Jul 31, 2019

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

@luizjr
Copy link
Author

luizjr commented Jul 31, 2019

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
  },
};

@nidib
Copy link

nidib commented Jul 31, 2019

#22. Foi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants