Bonandry / adwaita-plus

GNOME++, a third-party icons theme, based on new GNOME 3.32's Adwaita
GNU General Public License v3.0
163 stars 7 forks source link

Bug in Folders - Dark Theme #19

Closed btd1337 closed 5 years ago

btd1337 commented 5 years ago

When the theme Dark is set, see like are the folders:

Captura de tela de 2019-06-05 22-25-17

gusbemacbe commented 5 years ago

Olá @btd1337 !

Andrea não faz temas escuros. Quem assume a construção de temas escuros sou eu. Mas se estás muito apressado, preparo-te os comandos:

cd GNOME++-Dark
sed --in-place --follow-symlinks 's/#e6ddff\|#241f31/#444444/g' **/places/*.svg
btd1337 commented 5 years ago

Correção do comando:

sed --in-place --follow-symlinks 's/#e6ddff\|#241f31/#444444/g' ./places/**/*.svg

PS: A pasta places/scalable está faltando, vocês usam symlink pro tema padrão?

Captura de tela de 2019-06-06 02-57-55

gusbemacbe commented 5 years ago

PS: A pasta places/scalable está faltando, vocês usam symlink pro tema padrão?

Agora estou ciente de a pasta scalable estar ausente no GNOME++ Escuro, vou corrigir. Se estás apressado, executa.

ln -sfnr GNOME++/apps/scalable GNOME++-Dark/apps/scalable

Observa que GNOME++ Escuro é designado para o ambiente plenamente escuro. Para ambiente claro mas com panéis e barras de lado escuras, como o caso do Nautilus com tema de GTK Telinkrin, então primeiramente tenho de conversar com o Andrea se ele me deixa fazer para as distribuições não-Ubuntu (por exemplo, elementary OS e Linux Mint).

gusbemacbe commented 5 years ago

Não, errei, vi que a pasta scalable está presente là. Sim, utilzamos ligações simbólicas. Mas vou verificar.

btd1337 commented 5 years ago

@gusbemacbe Aqui não está presente: https://github.com/Bonandry/gnome-plus/tree/master/GNOME%2B%2B-Dark/places

gusbemacbe commented 5 years ago

Muito obrigado por ter conseguido descobrir. Pensei que fosse no apps. No fim de semana, corrigirei e faremos mais íconos porque estou a tentar resolver os problemas no Arch Linux.

Não sei se ainda estás a utilizar Arch Linux porque tu me reportaste um problema de Suru++ Telinkrin no Arch Linux. Se sim, dá uma olhada: https://plus.diolinux.com.br/t/habilitar-kms-para-as-placas-intel-aceleracao-no-chromium-e-habilitar-o-modulo-i915-mas-quero-saber-se-o-que-estou-a-fazer-e-certo-ou-errado/5047

gusbemacbe commented 5 years ago

Pronto, @btd1337 !

Por favor, transfere a branch beta, testa com novos ícones e com nova pasta places/scalable. Caso funcione, avisa-me que vou fazer pull request da branch beta para branch official.

btd1337 commented 5 years ago

As pastas foram corrigidas. Percebi que alguns ícones também estão ausentes na versão dark, por exemplo o da Lixeira, dentre outros. Poderia usar symlink da pasta de Apps entre a versão dark e a original.

gusbemacbe commented 5 years ago

Vou investigar.

gusbemacbe commented 5 years ago

Desuclpe-me pela minha ausência, é que eu tinha estado muito ocupado e tinha de esperar receber o meu novo monitor.

Todos os ícones de lixeira foram adicionados a ambos os temas normal e escuro, mas não sei quais outros ícones tu disseste.

btd1337 commented 5 years ago

Estes ícones simbólicos devem ser rosa mesmo?

Estão do mesmo jeito que as pastas...

Captura de tela de 2019-06-22 13-21-02

gusbemacbe commented 5 years ago

Vou ver por que o tema não está a utilizar os ícones coloridos nessa janela pop-up. Estás a utilizar Unity no Arch Linux?

gusbemacbe commented 5 years ago

Concluí que o problema está no index.theme. Vou testar com o index.theme do Suru++.

gusbemacbe commented 5 years ago

Então a culpa não foi do index.theme e sim das heranças das pastas. As pastas 24e 32 herdam da pasta 16, @btd1337.

Mas tenho de pedir autorização para @Bonandry se posso herdar novamente a pasta scalable às pastas 24 e 32 e criar a nova pasta 22.

gusbemacbe commented 5 years ago

Ciao @Bonandry !

Vedi l'ultima immagine, perché le cartelle 24 e 32 sono ereditate dalla cartella 16, e non dalla cartella scalable. Posso ereditare nuovamente le cartelle 24 e 32 alla cartella scalable e creare la nuova cartella 22? Solamente nelle cartelle apps, mimetypes e places. E posso correggere il file index.theme?

Bonandry commented 5 years ago

Ciao @gusbemacbe, certo

btd1337 commented 5 years ago

@gusbemacbe Não uso Unity. Uso o Gnome.

gusbemacbe commented 5 years ago

@btd1337, OK, como @Bonandry me autorizou, vou corrigir, mas primeiramente vou fazer os ícones que acabaste de pedir noutro tópico.

gusbemacbe commented 5 years ago

Pronto, por favor, testa.

btd1337 commented 5 years ago

@gusbemacbe Acredito que o problema não é este que você tem mente. Aparentemente o tema foi projetado para ter ícones simbólicos nestes casos. Veja que em todos os outros casos o problema continua:

Captura de tela de 2019-06-25 13-35-26

Captura de tela de 2019-06-25 13-34-56

Captura de tela de 2019-06-25 13-39-00

Captura de tela de 2019-06-25 13-44-33

A solução que usou, realmente usa os ícones maiores, mas veja como isso destoa do restante do tema e não fica bom. Os ícones ficam grandes demais e o contexto de todos os outros lugares continua usando os simbólicos.

Captura de tela de 2019-06-25 13-40-19

Acredito que o correto é aplicar uma cor escura a estes ícones que até então estão com a cor rosa. E se possível desfazer a mudança nas pastas 24 e 32, voltando a usar ícones no estilo simbólico.

gusbemacbe commented 5 years ago

Oi, @btd1337! Ainda estou confuso. Qual um dos três temas da família GNOME++: GNOME++, GNOME++ Escuro ou GNOME++ Colorido?

btd1337 commented 5 years ago

A issue é sobre o tema escuro, todas as imagens são do tema escuro. Somente na janela ABRIR COM teve alteração pros ícones escaláveis, devido a última modificação que fez. Mas não ficou bom. Acredito que a solução correta é fazer os ícones simbólicos serem pretos ao invés de rosa.

gusbemacbe commented 5 years ago

Oi @btd1337 !

O GNOME++ Escuro é designado para utilizadores do ambiente gráfico cheiamente escuro. Por isso, criei GNOME++ Claro, inspirado pelo Papirus Light do @SmartFinn hoje.

Testa se GNOME++ Claro é o que tu queres excatamente.

btd1337 commented 5 years ago

@gusbemacbe acertou em cheio! Era isso mesmo que eu tinha em mente. Muito obrigado! Parabéns pelo seu ótimo serviço!

gusbemacbe commented 5 years ago

óptimo, vou finalizar o projecto e lançá-lo hoje.

gusbemacbe commented 5 years ago

@btd1337 , por favor, utiliza o teu projecto para enviar a lista de ícones perdidos no meu projecto Suru++.