guardian/scribe

br tags in headings

Open

#481 aperta il 7 giu 2016

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (265 fork)batch import
bughelp wanted

Metriche repository

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

Descrizione

It seems like <br> in h1/h2/h3 etc. tags makes document.execCommand('formatBlock', false, 'p') function work not the way it have to.

Way to reproduce bug at the demo-page:

  1. create three <p>-paragraphs
  2. select them
  3. click h2-button, so you get <h2>Hello, World!<br>Hello, World!<br>Hello, World!</h2>
  4. set caret after last <br> and try to click h2-button again to remove h2 tag. It will not work.
  5. If you wil set caret before <br> and try to click h2, at first it will remove <br>-tag and after another click it will acts like it have to.

Guida contributor

Direzione di ricerca
Esamina l'implementazione di formatBlock nel codice, concentrandoti su come vengono gestiti gli elementi <br> all'interno delle intestazioni. Traccia la logica quando formatBlock viene chiamato con 'p' per rimuovere l'intestazione. Verifica se la presenza di <br> causa una risoluzione errata della selezione.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75