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

Line numbers not applied to all Questions #167

Open
danopolan opened this issue Jan 14, 2025 · 3 comments · May be fixed by #169
Open

Line numbers not applied to all Questions #167

danopolan opened this issue Jan 14, 2025 · 3 comments · May be fixed by #169
Assignees

Comments

@danopolan
Copy link
Contributor

Some questions in CTAL-TA Sample Exam Questions document are skipped for line numbering.
Answers table has no line numbers, but I assume that !54 comment still stands.

Build: https://github.com/istqborg/istqb-ctal-ta/actions/runs/12776584329

image
@Witiko
Copy link
Contributor

Witiko commented Jan 15, 2025

I don't think robust support for line numbering in LaTeX will be possible in the near future, since LaTeX is quite basic compared to e.g. HTML and lacks the necessary abstractions such as the DOM to allow similar reflection.

Regardless, I have some hunches about fixing this particular issue. Let me look into those.

@Witiko
Copy link
Contributor

Witiko commented Jan 17, 2025

@danopolan This is caused by the code from PR #38 that prevents splitting short questions over more pages.

If you'd like, we can disable the code when line numbers are enabled. The downside is that pages will break differently for reviewers, which increases the risk that an issue with page breaking would go unnoticed during review.

Alternatively, we might also disable to code altogether. This will ensure that pages are broken the same for reviewers. However, short questions may be split over more pages.

@Witiko
Copy link
Contributor

Witiko commented Jan 17, 2025

If you'd like, we can disable the code when line numbers are enabled. The downside is that pages will break differently for reviewers, which increases the risk that an issue with page breaking would go unnoticed during review.

I opened PR #169, which implements this change.

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 a pull request may close this issue.

2 participants