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

Config panel #6

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

Config panel #6

wants to merge 10 commits into from

Conversation

diglopes
Copy link

@diglopes diglopes commented Jun 1, 2020

Painel de configuração

Problematica

  • Necessidade não muito intuitiva de editar arquivos dentro do pŕoprio projeto para alterar os textos exibidos na tela.
  • Alteração do timer por query params não muito intuitiva
  • Não possui uma fácil depuração da aparência dos textos em tela de forma instantânea.

Proposta de solução

Esse pull request implementa um painel de configuração que permite editar tanto os textos que serão exibidos quanto o tempo da contagem regressiva de forma mais simples, eliminando a necessidade de editar arquivos de texto dentro do pŕoprio projeto ou enviar queries na url. As configurações passam a ser feitas no proprio front-end com as informações sendo salvas e recuperadas no localstorage.

Utilizando

Para abrir o painel basta pressionar CTRL+ALT+E na tela do projeto que irá abri-lo, para fechar pasta realizar o mesmo comando ou pressionar ESC.

@fernanDOTdo
Copy link
Member

Opa! Estávamos olhando o seu Pull Request na live e todos nós achamos a ideia muito boa!
Antes de aceitar o PR, eu gostaria de saber se você testou suas alterações no OBS. Porque atualmente nós usamos arquivos externos e Query String Parameters porque normalmente, depois que o countdown está configurado no OBS, nós não o acessamos mais via navegador.
Minha principal dúvida é se eu vou conseguir usar o mesmo Local Storage definido no browser também no OBS.
Se você não testou, eu mesmo vou fazer um teste, mas queria ouvir sua opinião primeiro.

@diglopes
Copy link
Author

Opa! Estávamos olhando o seu Pull Request na live e todos nós achamos a ideia muito boa!
Antes de aceitar o PR, eu gostaria de saber se você testou suas alterações no OBS. Porque atualmente nós usamos arquivos externos e Query String Parameters porque normalmente, depois que o countdown está configurado no OBS, nós não o acessamos mais via navegador.
Minha principal dúvida é se eu vou conseguir usar o mesmo Local Storage definido no browser também no OBS.
Se você não testou, eu mesmo vou fazer um teste, mas queria ouvir sua opinião primeiro.

BANG! Não testei antes de mandar, na verdade errei em não verificar como funcionava o OBS para esse tipo de recurso 😢 ... e realmente isso gera um problema na hora que apresentar as informações, teria que utilizar uma outra fonte para fazer o gerenciamento desses dados

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.

2 participants