search

rafatosta / zapzap

WhatsApp desktop application written in Pyqt6 + PyQt6-WebEngine.
https://rtosta.com/zapzap-web/
GNU General Public License v3.0
86 stars 2 forks source link

Close Conversation shortcut #104

Closed RoinujNosde closed 2 years ago

rafatosta commented 2 years ago

Gostei, ficou muito bom. Parabéns. Vou fazer um ajuste dessa funcionalidade para que seja executada ao Esc ou ao fechar a janela para que rode em 2o plano e para todas as janelas ativas do app.

rafatosta commented 2 years ago

Notei que Grupos não possuem essa opção. Poderia colocar alguma condicional para que se não tenha o item o popup seja fechado? image

RoinujNosde commented 2 years ago

Notei que Grupos não possuem essa opção. Poderia colocar alguma condicional para que se não tenha o item o popup seja fechado? image

https://github.com/rafatosta/zapzap/pull/104/files#diff-7754032c785c1ecb88a8bb6cb55b809abaeb42028fe269bd69ac2f9bc5f6dab4R118 Essa linha deveria ter feito isso.

rafatosta commented 2 years ago

Notei que Grupos não possuem essa opção. Poderia colocar alguma condicional para que se não tenha o item o popup seja fechado? image

https://github.com/rafatosta/zapzap/pull/104/files#diff-7754032c785c1ecb88a8bb6cb55b809abaeb42028fe269bd69ac2f9bc5f6dab4R118 Essa linha deveria ter feito isso.

Pois é, acabei de ver isso aqui no código. Também não entendi porque não funcionou

RoinujNosde commented 2 years ago

Mais tarde dou uma olhada. Talvez precise de um setTimeout, ou de simular o evento do ESC sendo pressionado.

Também tá faltando uma string nos arquivos de linguagem para essa ação.

rafatosta commented 2 years ago

Mais tarde dou uma olhada. Talvez precise de um setTimeout, ou de simular o evento do ESC sendo pressionado.

Também tá faltando uma string nos arquivos de linguagem para essa ação.

O evento do Esc já está presente no app e não pode ser aplicado no JS. Quando menos loop no JS melhor, ou até nenhum, pois derruba o desempenho do app. Só verifica isso de não está fechando o menu se a opção não for encontrada.

Pode deixar que essa parte eu coloco no código, pois o Esc é utilizado para sair do painel das Configurações. Então é preciso verificar em qual painel está, se tiver nas configurações volta pra o home, caso contrário fecha a conversa atual.

rafatosta commented 2 years ago

Sobre "Também tá faltando uma string nos arquivos de linguagem para essa ação.", não se preocupa com isso agora. Pretendo colocar uma tabela com os atalhos disponíveis para facilitar para o usuário, pois o menu não é mostrado se usar a decoração do app, apenas com a decoração do sistema.

rafatosta commented 2 years ago

@RoinujNosde verificou se continua aberto a janela quando não há a opção? Pretendo lançar uma versão semana que vem para corrigir algumas traduções e já mandava isso junto.

RoinujNosde commented 2 years ago

Andei testando algumas coisas... mas sem muita sorte. Só recarregando a página com location.reload().

RoinujNosde commented 2 years ago

@rafatosta testa o último commit, me diz o que acha. Só preciso remover a "sombra" que fica no botão de opções: image

rafatosta commented 2 years ago

@rafatosta testa o último commit, me diz o que acha. Só preciso remover a "sombra" que fica no botão de opções: image

Acredito que esse detalhe seja aceitável. Ficou muito bom!

RoinujNosde commented 2 years ago

Beleza. Se eu pensar em alguma forma de remover a sombra, faço outra PR.

rafatosta commented 2 years ago

Implementei usando o Esc ou se a janela for fechada. Mudei o script de lugar, para o whatsapp.py, já que é de responsabilidade dele. E mais algumas coisas por causa do multi usuário..

Vou fechar esse PR e o que você mandar será apenas no JS e só faço aceitar.