shutterstock/rickshaw

Use of pointer-events: none; on non-svg items

Open

#84 aperta il 13 lug 2012

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (1009 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (6540 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

In detail.css, you apply pointer-events: none; on .rickshaw_graph .detail to ensure that tooltips don't get hovered over and thus don't prevent other tooltips from popping up..

I find that there are two issues with that:

  1. pointer-events: none; shouldn't really be used for non-svg items as described in the spec
  2. it's impossible to hover over the tooltip itself, which might be nice if there is a link in there or something like that..

In my opinion, the approach of disabling pointer events and setting opacity: 0 to all tooltips would work much nicer if you enabled pointer events and then replace the opacity switch with a display: none/inline switch..

Works like a charm for me and everything is as expected

Cheers

Guida contributor

Direzione di ricerca
Sostituire pointer events: none con un interruttore display: none/inline per i tooltip in detail.css.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
CSS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85