feat(submit-hotkey): submit post with Ctrl (or Command) + Enter
#490
Conversation
|
@aprendendofelipe is attempting to deploy a commit to the TabNews Team on Vercel. To accomplish this, @aprendendofelipe needs to request access to the Team. Afterwards, an owner of the Team is required to accept their membership request. If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account. |
Testado em Mac, está funcionando! 👍 |
|
Show @vinicius-arcanjo, muito obrigado! |
filipedeschamps
force-pushed
the
submit-with-hotkeys
branch
from
4c53ad1
to
c90eb8d
Compare
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@aprendendofelipe fiz rebase e um pequeno refactor renomeando de Está deployed em https://tabnews-git-submit-with-hotkeys-tabnews.vercel.app/ e pelo que testei aqui no Mac também está funcionando, testei criar posts, respostas e atualizar, tudo 100% e que delicinha! O único bug que eu peguei, e que não sei se vale a pena bloquear esse merge se for difícil de consertar, é que se você tem duas caixas abertas de edição dentro da mesma página e usar o atalho, as duas são enviadas ao mesmo tempo. |
Amanhã vejo se consigo fazer algo para submeter somente o post ativo. Talvez o bytemd dificulte isso, mas vale a pena tentar 👍 |
aprendendofelipe
force-pushed
the
submit-with-hotkeys
branch
from
c90eb8d
to
4110067
Compare
Quem disse que dá pra dormir sem ficar pensando em uma solução? Mais fácil já resolver logo e dormir tranquilo... hahaha |
hahahahahah entendo perfeitamente! 😂 😂 😂 😂 Merged!!!! Let's gooooooo!!!!! |
|
Hmm que estranho, não teve commit do merge, e a Vercel não disparou o deploy por conta disso. |
Eu fiz rebase localmente antes de dar push... Daí não deve ter precisado do commit de merge. Acabei de fazer a mesma coisa no #494 |
|
Foi eu quem vacilou aqui na interface do GitHub, eu tinha deixado uma outra opção de merge (squash) por conta de outro PR, percebi agora. Mas não tem problema, já foi feito o deploy por conta de outro merge que fiz agora 🤝 e testei em produção o editar de uma publicação minha e funcionou (inclusive, tentando forçar o bug)... tudo 100% 🎉 |
|
@aprendendofelipe esse PR e todos os outros que entraram na |
Como sugerido em #382.
O editor do bytemd não expõe um
onKeyDownnem nada parecido, então vamos deaddEventListener("keydown", onKeyDown)para poder submeter um conteúdo ou comentário com os atalhosCtrl+EnterouCommand+Enter.Preciso que alguém teste no Mac.