Skip to content
This repository has been archived by the owner on Feb 27, 2024. It is now read-only.

glossaryText field should put all text on one line, and not use \-characters as line breaks. #43

Open
RieksJ opened this issue Oct 19, 2022 · 3 comments
Labels
bug Something isn't working prio=high

Comments

@RieksJ
Copy link
Contributor

RieksJ commented Oct 19, 2022

The glossaryText field is not on one line, but contains -characters that designate line breaks.

Thus, if such a line were to include a lengthy termref such as this is a very long showtext, it could be spread over two lines, which would mean that the TRRT might not recognize it as the termref that it is.

@morethinks
Copy link

Created a PR for this issue. glossaryText wasn't the only field having this problem.
#46

@brianorwhatever
Copy link
Contributor

@RieksJ when you say "The glossaryText field is not on one line" do you mean in the test files or is there a specific location you are referring to? It appears Ryan has fixed this in the test files but before closing this would like to confirm that is the correct location.

@RieksJ
Copy link
Contributor Author

RieksJ commented Mar 3, 2023

Back in October, I used the MRG generator (locally) on the 'curated texts' of the TEv2 documentation. The result is here. Lines 63, 91-92, and more, show what's wrong.

I can imagine though that it is not just the glossaryText field that would suffer from broken lines. I expect that there may be other fields that have lengthy texts. The terms in the essif-lab framework repo, for example, have fields 'hoverText' that will also be quite long.

I am under the impression that MRG-entries that are created according to the current version of the spec would not have such issues. Could you make sure this is the case?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working prio=high
Projects
None yet
Development

No branches or pull requests

2 participants