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

Fix #52 adoc-nobreak-p checks adoc-nobreak text property #53

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TobiasZawada
Copy link
Collaborator

Fix #52 adoc-nobreak-p checks adoc-nobreak text property

@TobiasZawada
Copy link
Collaborator Author

TobiasZawada commented Feb 10, 2024

Same thing like Flyspell-Overlays (see #50).

xref:LINK[LINK TEXT] is open for discussion. Asciidoctor actually accepts linebreaks in LINK TEXT but adoc-mode does not font-lock it right, since it is not multiline. (Multiline font lock is quite tricky and needs thorough work. (E.g., two-line titles also do not work very well.)

The current solution is to not let auto-fill-mode introduce artificial linebreaks into LINK TEXT. But, that is a hack.

@TobiasZawada
Copy link
Collaborator Author

It is a pity, but I have the impression that we need to remove the marking of two-line headers.
The adoc-nobreak property spills out on the text of the following paragraph. (To be investigated.)

@bbatsov
Copy link
Owner

bbatsov commented Feb 18, 2024

It is a pity, but I have the impression that we need to remove the marking of two-line headers.

That'd be fine by me, but we should mention this somewhere and encourage people to avoid using those.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

auto-fill-mode breaks one-line section titles
2 participants