guardian/scribe

br tags in headings

Open

#481 ouverte le 7 juin 2016

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (265 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (3 516 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez l'implémentation de formatBlock dans le code, en vous concentrant sur la gestion des éléments <br> dans les titres. Suivez la logique lorsque formatBlock est appelé avec 'p' pour supprimer le titre. Vérifiez si la présence de <br> provoque une mauvaise résolution de la sélection.
Stack technique
javascript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75