Adiciona ícone de comentário e atualiza readme

[claromes]

Além do ícone, também deixei somente um readme

Preview:

[rghedin]

Vi que usou svg inline. O @altendorfme comentou que, para fins de cache, é melhor usá-lo como arquivo. (E, como efeito colateral, fica mais fácil modificar o tamanho assim.)

Rola fazer essa mudança? Coloca o arquivo *.svg dentro do diretório /img/. E acho que dá para diminuir um pouco o tamanho do ícone no site, algo mais alinhado ao tamanho do número de comentários.

[claromes]

OK. Eu vou segurar essa mudança pq estou testando outros formatos de ícone no Órbita... E preciso mudar tbm a função que retorna a contagem, pra não repetir código. Deixei o ícone maior pq se não o "rabinho" do balão some... No formato mais arredondado fica mais alinhado (vou mandar um teste do Órbita no matrix)

[rghedin]

Não sei se rola alguma implicação em acessibilidade ou outro aspecto, mas uma alternativa seria colocar o *.svg como background-image da classe comments-link. Salvo engano, é mais fácil controlar o posicionamento e evitar esses cortes chatos que acontecem com imagens inline.

[altendorfme]

@claromes Recomendo passar o SVG no https://jakearchibald.github.io/svgomg/ também, ele remove os comentários e alguns sujeirinhas pra deixar mais leve, eu vi que tá usando FontAwesome, ai tem que desmarcar o comentário de license (@rghedin) deles ou trocar pelo https://icons.getbootstrap.com/ se for viável

@rghedin um trick do SVG é que pode usar o aria-label dentro do SVG, tipo:

<svg ... role="img" aria-label="Comentário">
...
</svg>

[claromes]

@rghedin Mantive a lógica do Órbita:

[rghedin]

Implementei em produção, @claromes.

Achei que o ícone de comentários ficou muito discreto ali embaixo. Tem um ajuste simples que, acho, deixa ele mais destacado: colocar acima do título, junto à hora de publicação e autor(a):

Outra coisa, que devia ter dito antes, é talvez puxar o ícone via CSS, usando background-image sem repetição. Creio que seja melhor do que puxar várias vezes uma imagem dentro do loop.

O que acha?

Vou tentar escrever essas mudanças, aí faço um commit neste mesmo branch.

[rghedin]

Acabei usando a versão com stroke para tudo. A cheia ficou “pesada” naquele espaço. Dá uma olhada lá no site (está em produção):

Se não tiver objeção, acho que podemos usar esse mesmo modelo no Órbita.

[claromes]

@rghedin Ficou ótimo! Mas na versão dark o filter não está sendo aplicado (incluindo no logo). Pro Órbita acho ok tbm.

[rghedin]

Ops, sempre esqueço do modo noturno! Vou corrigir.

[rghedin]

Tive que colocar um <span> no número do comentário para conciliar os filtros invert(), mas deu certo.

[claromes]

Assim que fizer o commit com as alterações, eu altero as imagens pra bg no css

[claromes]

@rghedin Ta td certo aqui. Mas você precisa aceitar a revisão e aí fazer o merge, pois eu que abri o PR :D

[rghedin]

Ótimo!

Eu pequei outra vez e fiz um monte de alterações no tema em produção. (Pareço viciado falando que paro quando quiser.) Vou aceitar este PR e depois fazer um PRzão para sincronizar produção e repositório, ok?