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

Seletor de arquivo para upload não reflete tema da distro #83

Closed eduardomozart closed 2 years ago

eduardomozart commented 2 years ago

Boa noite, Rafael Apenas uma dúvida. Abri um Bug Report lá na página da Distro (https://github.com/linuxdeepin/developer-center/issues/3357), mas talvez você já tenha encontrado esse erro antes. Compilei o ZapZap através do código fonte, mas ao selecionar um arquivo para upload, o tema do seletor de arquivo não corresponde ao do sistema operacional. Uso o tema Dark e a janela de seleção de arquivo é White. Você já encontrou esse erro antes? Tem algo que possamos definir nos argumentos através do Flatseal para forçar o tema Dark? O próprio App tem o seletor de tema, selecionei o tema Dark, mas aparentemente ele não está aplicando no File Chooser do PyQt. Capturar_zapzap_20220802212805

rafatosta commented 2 years ago

O app não controla o tema das aplicações externas, nem do seletor de arquivo e não há nenhuma "informação" sobre tema passado, apenas pede para o sistema que abra o seletor de arquivo através do QFileChooser.

rafatosta commented 2 years ago

Problema da distro, Ver #76

herzenschein commented 2 years ago

Este erro ocorre no Ubuntu também. Creio que descobri a razão.

Explicando do zero:

Este mesmo problema também ocorre com o Telegram, que migrou pro Qt6.

O ZapZap exibe a janela de diálogo correta no meu Fedora Rawhide (37), mas não em distros como as versões do Ubuntu que não tem Qt6.

Solução temporária: para manualmente forçar o uso do portal para utilizar o tema correto, use:

QT_QPA_PLATFORMTHEME=flatpak flatpak run com.rtosta.zapzap

Ou melhor ainda, salve a variável de ambiente para não ter que rodar toda vez:

flatpak override --env="QT_QPA_PLATFORMTHEME=flatpak" com.rtosta.zapzap

(precisa de sudo caso vc tenha instalado o flathub sem --user)

rafatosta commented 2 years ago

@herzenschein O que acontece com a definição dessa variável em distribuições que estão funcionando? Funciona adicionando pelo Flatseal, correto? Se não afetar posso torná-la padrão.

herzenschein commented 2 years ago

Pelo que testei, não parece causar nenhum problema, continua usando o portal do mesmo modo.

Posso perguntar lá no grupo do #flatpak no Matrix depois.

herzenschein commented 2 years ago

Aliás, já que já está no flathub, deve ser possível criar um issue por lá e pingar o pessoal do flatpak por lá para confirmar, tipo o hfiguiere, barthalion ou o nedrichards.