palantir/blueprint

Using EditableText in selected tree element leads to unreadable text

Open

#2.236 aberto em 14 de mar. de 2018

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)TypeScript (2.167 forks)batch import
Package: coreType: bughelp wanted

Métricas do repositório

Stars
 (20.263 stars)
Métricas de merge de PR
 (Mesclagem média 43d 10h) (27 fundiu PRs em 30d)

Description

Bug report

  • Package version(s):
    "@blueprintjs/core": "^2.0.0-rc.2",
    "@blueprintjs/icons": "^2.0.0-rc.2",
    "@blueprintjs/table": "^2.0.0-rc.2",
  • Browser and OS versions: Chrome 64.0.3282.186 on OS X

Steps to reproduce

  1. Create
  2. Add child element with EditableText as label
  3. Select element and start editing

Actual behavior

The editable text is unreadable because it's white text on white background.

style-problem

Expected behavior

Visibly different colors for text and background.

Shouldn't pt.editable-text.pt-editable-editing set the text-color when setting the background-color? Setting only one of them sounds like inviting this kind of problems.

Guia do colaborador

Direção de pesquisa
Investigue a regra CSS para .pt editable text.pt editable editing e garanta que ela defina tanto a cor de fundo quanto a cor do texto quando o nó da árvore estiver selecionado.
Pilha de tecnologia
typescriptreactcss
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