shutterstock/rickshaw

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

Open

#84 aberto em 13 de jul. de 2012

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)JavaScript (1.009 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (6.540 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Substitua pointer events: none por um interruptor display: none/inline para os tooltips em detail.css.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
CSS
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85