loopinfinito / setup

Loop Infinito's Setup website
http://setup.loopinfinito.com.br
23 stars 11 forks source link

Adicionando target=_blank nos links dentro dos textos de respostas #48

Closed danielGalvao closed 9 years ago

caiogondim commented 9 years ago

@almirfilho O que tu acha?

almirfilho commented 9 years ago

Caramba, juro que nem tinha visto! Valeu pela contribuição, @danielGalvao! Acho uma boa ideia. Porém, acho que tem como melhorar um pouco a solução proposta. Pode-se utilizar o querySelectorAll() e assim deixar de usar fors aninhados ;)

caiogondim commented 9 years ago

Precisamos fazer isso via JS? Não seria melhor por direto no HTML?

danielGalvao commented 9 years ago

O problema é que o markdown parece que mesmo colocando link não vira target=_blank. Ou vira?

danielGalvao commented 9 years ago

Eu também não tinha visto o arquivo main.js.coffee, não manjo de coffeescript mas posso tentar adaptar para querySelectorAll() por lá Almir. Como colocaria no HTML Caio?

caiogondim commented 9 years ago

Apesar de termos vários sites com esse comportamento (inclusive a empresa que trabalho recentemente usa muito disto), eu acho que isto é mais para usuários leigos.

Eu particularmente prefiro que uma aba seja aberta quando eu quero. Então eu só seguro a tecla cmd e aperto o link, para abrir em uma nova aba.

@almirfilho O que acha?

almirfilho commented 9 years ago

Sim, mas pelo que eu entendi por 'textos de respostas', seria nos links dos comentários, certo?
O problema é que o Disqus monta seu conteúdo num iframe. Também não consegui encontrar nenhum elemento com a class .question-content como mostrada no código.

@danielGalvao você testou isso? Funcionou localmente pra você?

danielGalvao commented 9 years ago

@almirfilho Eu testei sim, mas não são nos comentários, não tem comentário no Setup de cada entrevistado. No caso a dificuldade que eu tenho é que muitas vezes estou lendo os textos dos entrevistados e eles apresentam diversas ferramentas, frameworks, Apps, etc em links, com isso ao clicar nesses links eu era jogado em outra página e muitas vezes eu faço outras coisas enquanto leio. Isso quebrava toda a leitura. O elemento com a class .question-content é uma div que está contida em

section.question > div.wrapper > div.question-content-area

almirfilho commented 9 years ago

@danielGalvao cara, foi mal.. não sei porque, mas pensava que era no Loop Infinito! Tava procurando no lugar errado! haha

Mas realmente, não há maneira de fazer via markdown. Teria que ser com JS mesmo.

danielGalvao commented 9 years ago

@almirfilho Relaxa. Mas assim, se como o @caiogondim comentou que não é o perfil dos usuários, não sendo necessário essa mudança, vamos declinar. Eu deveria ter aberto uma issue antes do pull request...

caiogondim commented 9 years ago

Sem problemas. Valeu pelo tempo e contribuição =)