Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Utilise la hauteur de ligne par défaut dans les <textarea> #4601

Conversation

motet-a
Copy link
Contributor

@motet-a motet-a commented Aug 31, 2017

La version de normalize.css utilisée règle line-height: 1.15 sur les

<textarea>. Cela ne marche pas bien avec certaines polices, causant des chevauchements des sélections entre les lignes. https://github.com/necolas/normalize.css/blob/7.0.0/normalize.css#L247 Fix #4590. | Q | R | ----------------------------------- | ------------------------------------------- | Type de modification | correction de bug | Ticket(s) (_issue(s)_) concerné(s) | #4590 ### QA Vérifiez que quand vous sélectionnez des choses dans l’éditeur, les lignes ne se chevauchent pas (voir issue).

@motet-a motet-a added C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant labels Aug 31, 2017
@motet-a motet-a added this to the v25 milestone Aug 31, 2017
@motet-a motet-a self-assigned this Aug 31, 2017
@motet-a motet-a changed the title Utilise la hauteur de ligne par défault dans les <textarea> Utilise la hauteur de ligne par défaut dans les <textarea> Aug 31, 2017
La version de normalize.css utilisée règle `line-height: 1.15` sur les
<textarea>. Cela ne marche pas bien avec certaines polices, causant
des chevauchements des sélections entre les lignes.

https://github.com/necolas/normalize.css/blob/7.0.0/normalize.css#L247

Fix zestedesavoir#4590.
@motet-a motet-a force-pushed the interligne-messages-regression-maj-normalize branch from c804114 to 5c98eda Compare August 31, 2017 09:33
@motet-a motet-a added the QA svp label Aug 31, 2017
@coveralls
Copy link

coveralls commented Aug 31, 2017

Coverage Status

Coverage remained the same at 89.459% when pulling 5c98eda on motet-a:interligne-messages-regression-maj-normalize into d873968 on zestedesavoir:release-v25.

@artragis artragis self-assigned this Aug 31, 2017
@artragis artragis removed the QA svp label Aug 31, 2017
@artragis
Copy link
Member

QA OK

@artragis artragis merged commit bdb747a into zestedesavoir:release-v25 Aug 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants