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

New Lines in Messages #771

Merged
merged 5 commits into from
Sep 8, 2024
Merged

Conversation

BeckyDTP
Copy link
Contributor

@BeckyDTP BeckyDTP commented Sep 3, 2024

Since the \n character did not create a new line in the messages, I added <br> in three places. This works well.
Below is the before and after view.

Message One:
sigil-no-valid-spine-before
sigil-no-valid-spine-after

Message Two:
sigil-no-identify-ncx-before
sigil-no-identify-ncx-after

Message Three:
sigil-no-ncx-before
sigil-no-ncx-after

@kevinhendricks
Copy link
Contributor

Thank you. These make good sense. I will make these changes after the upcoming fast follow-up bug fix release for version 2.3.1 which we hope to happen in a week or so. If we add them now, that will necessitate changes to translations that would delay the release while we wait for volunteers to handle them.

@kevinhendricks
Copy link
Contributor

BeckyEbook, there are conflicts with the late changes I made to improve missing NCX handling in ImportEPUB.cpp and this PR. Would you please rebase your message changes from this PR on the released 2.3.1 source, so I can merge it without any conflicts.

Thanks!

@kevinhendricks kevinhendricks merged commit 6992078 into Sigil-Ebook:master Sep 8, 2024
4 checks passed
@kevinhendricks
Copy link
Contributor

Thank you!

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.

2 participants