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/preferences window #22

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

Feature/preferences window #22

wants to merge 5 commits into from

Conversation

nidib
Copy link

@nidib nidib commented Jul 31, 2019

Adiciona window para settings do usuário.

image

@nidib nidib mentioned this pull request Jul 31, 2019
@diego3g
Copy link
Member

diego3g commented Aug 5, 2019

Podes corrigir o conflito com o arquivo main.js originado da PR que aceite há pouco? Além disso, uma boa dica é colocar arquivos que são do sistema que você quer ignorar em um arquivo global de ignore do git ao invés de ser por projeto, como o seu .DS_Store (https://gist.github.com/subfuzion/db7f57fff2fb6998a16c).

Você fez apenas a página de preferências, certo? Se eu aceitar essa PR ficamos ainda sem a parte das preferências funcionarem.

Acho legal a PR incluir toda parte de preferências dentro dela, o funcionamento também, pode deixar por enquanto só a parte de iniciar ou não quando o computador liga, não precisa as outras por enquanto.

@nidib
Copy link
Author

nidib commented Aug 5, 2019

Podes corrigir o conflito com o arquivo main.js originado da PR que aceite há pouco? Além disso, uma boa dica é colocar arquivos que são do sistema que você quer ignorar em um arquivo global de ignore do git ao invés de ser por projeto, como o seu .DS_Store (https://gist.github.com/subfuzion/db7f57fff2fb6998a16c).

Você fez apenas a página de preferências, certo? Se eu aceitar essa PR ficamos ainda sem a parte das preferências funcionarem.

Acho legal a PR incluir toda parte de preferências dentro dela, o funcionamento também, pode deixar por enquanto só a parte de iniciar ou não quando o computador liga, não precisa as outras por enquanto.

Vou corrigir.. Quanto ao .DS_Store n sabia dessa opção, vou configurar aqui. Valeu! So fiz escrevi o html e css, n está funcional.

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