search

NextINpact / Next

18 stars 0 forks source link

Ouverture des nouveaux commentaires dans un nouvel onglet #155

Closed Wivik closed 8 months ago

Wivik commented 1 year ago

Habituellement, je fais un middle click pour ouvrir les nouveaux messages dans un nouvel onglet. Ici le bouton ne réagit pas à ce signal :'(

darkweizer commented 1 year ago

Bonjour,

Gros +1 ! Ce n'est plus une balise a mais une fonction js via onclick sur une balise span. D'où le fait qu'on ne puisse plus l'ouvrir avec un clique de mollette. Et pareil que vous, je navigue presque que comme ça dans les articles ici ☹

image

Wivik commented 1 year ago

A noter que ce comportement n'est présent que sur les articles principaux. Les bulles du brief sont bien middle-clickable.

pamputt commented 1 year ago

Ca ressemble beaucoup au ticket https://github.com/NextINpact/Next/issues/200. Doublon ?

darkweizer commented 1 year ago

Ca ressemble beaucoup au ticket #200. Doublon ?

Oui, le #200 est exactement le même que celui-ci !

oliverpool commented 1 year ago

A noter que ce comportement n'est présent que sur les articles principaux. Les bulles du brief sont bien middle-clickable

Dans la colonne de droite, toute la Brief est un seul lien (le titre et la bulle) qui renvoi vers l'article: image

Sur la page d'un Brief, le bug est bien aussi présent.

image

pipoupoip commented 11 months ago

Tous les boutons de commentaires sont alternativement cliquables :)

darkweizer commented 10 months ago

Tous les boutons de commentaires sont alternativement cliquables :)

C'est-à-dire ?! Actuellement, le clic molette ne fonctionne pas sur les icônes de commentaire sur les articles. Ce que demande le ticket

pamputt commented 10 months ago

Je viens de tester. Chez moi (Firefox sur ordinateur de bureau), le clic molette pour ouvrir dans un nouvel onglet fonctionne parfaitement, aussi bien en cliquant sur l'icone que sur le chiffre indiquant le nombre de commentaires.

darkweizer commented 10 months ago

Firefox également, ça fonctionne sur le bandeau droit du brief, pas sur le flux central de la homepage, ni sur la page du brief. Voir mon gif. Le clique molette fait apparaitre le défilement et non une nouvelle page (d'ailleurs quand on passe la souris sur les icônes, il n'y a pas l'url qui s'affiche en bas à gauche en prévisualisation). Pas de souci sur le bandeau de droite par contre. Brief comme flux d'actu courant. D'art d'art, c'est de l'art

C'est normal, on a toujours un span avec un onclick='goToComments()' pour les icônes sur le flux central et non un a. Là où tout l'article est a sur le bandeau droit (et ne redirige pas sur les commentaires, soit disant passant)

Ce ticket devrait être réouvert je pense (à moins qu'il y est une mise en prod non effectuée contenant le correctif)

dragonis41 commented 10 months ago

Ça marche très bien de mon côté (Chrome et Firefox). À tous ceux pour qui ça ne marche pas : êtes-vous sur Windows par hasard ?

oliverpool commented 10 months ago

Sous iOS/Safari, ça ne fonctionne pas.

Sous Linux/Firefox, ça fonctionne mal (les commentaires s'ouvrent dans un nouvel onglet, mais pas en arrière plan, comme les liens standards).

Pourquoi ne pas faire un lien standard <a href="...#comments> sur lequel la capture de clic a lieu (si c'est un clic normal, déclencher la logique JS, si c'est un clic du milieu, laisser faire le navigateur) ?

darkweizer commented 10 months ago

Ça marche très bien de mon côté (Chrome et Firefox). À tous ceux pour qui ça ne marche pas : êtes-vous sur Windows par hasard ?

Je confirme. Sur windows 11 Firefox 121.0.1 et windows 10 Chrome 120.0.6099.225.

(je peux tester ce soir sur un Windows 10 Firefox 121.0.1 s'il faut)

Alexandre-Gliganic commented 8 months ago

Bonjour,

Nous avons corrigé ce bug.

Merci du signalement :smile: