facebookarchive/draft-js

getCurrentInlineStyle behaves inconsistently

Open

#722 aberto em 17 de out. de 2016

Ver no GitHub
 (3 comments) (6 reactions) (0 assignees)JavaScript (2.646 forks)batch import
bughelp wanted

Métricas do repositório

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

Description

I believe there is a bug in getCurrentInlineStyle (or, if it's current behavior is intentional, it is inconsistent with typical rich text editor behavior).

When selecting a block of text in which different characters have different styles, getCurrentInlineStyle usually doesn't show styles that aren't applied to the entire selection. This seems to be standard practice. However, it will show styles that are applied to the very start of the selection, even if they aren't applied throughout, which seems inconsistent.

This can be observed in the sample editor at https://facebook.github.io/draft-js/ Enter : foo bar bat. Highlight the whole line and the bold button will not be lit. Highlight "foo bar" and, again, bold is unlit. Highlight "bar bat" and bold will light, despite the fact that all the characters in the selection are not bold.

I'm seeing this in version 0.9.1

Guia do colaborador

Direção de pesquisa
Investigue a função getCurrentInlineStyle no código fonte do Draft.js para entender como ela determina os estilos ativos para uma determinada seleção. Rastreie a lógica de quando um estilo é mostrado quando aplicado apenas a parte da seleção. Identifique onde ocorre a inconsistência (mostrar o estilo aplicado apenas no início) e proponha uma correção para torná lo consistente em toda a seleção.
Pilha de tecnologia
javascriptreact
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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
JavaScriptReactDraft.js
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70