Skip to content
This repository has been archived by the owner on May 5, 2021. It is now read-only.

[FIX] LineBreak: List item empty with br are displayed as twice line #456

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Gorash
Copy link
Contributor

@Gorash Gorash commented Oct 26, 2020

The mobile versions represent the
in li as making 2 lines and not
just one (unlike browsers). The implementation of linebreaks needs to
change. The invisible terminal BR must be an invisible character so that
the previous BR is displayed but respecting more the W3 rules.

@Zynton
Copy link
Contributor

Zynton commented Oct 27, 2020 via email

The mobile versions represent the <br> in li as making 2 lines and not
just one (unlike browsers). The implementation of linebreaks needs to
change. The invisible terminal BR must be an invisible character so that
the previous BR is displayed but respecting more the W3 rules.
@Gorash Gorash added discussion Extra discussion is needed invalid This doesn't seem right labels Oct 27, 2020
@Gorash
Copy link
Contributor Author

Gorash commented Oct 27, 2020

Parsing OK (font, container, empty...), rendering OK, but It's not good I think.

TODO later: Choice to be discussed because the <br> in the <li> whether it is for editing or not seems incorrect.
Therefore it is necessary to be able to add <p> in the li when there is at least one <br>.
Moreover, if there is a <p> during parsing, it must be kept.
(This is a bug regardless of the publisher and even without an editor)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
discussion Extra discussion is needed invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants