[Quickfix] fix Cloze question gap width - #12032
Open
kunzeus wants to merge 1 commit into
Open
Conversation
Contributor
|
hi @thojou could you please review as soon as possible? |
dsstrassner
added
the
css/html
See: https://mantis.ilias.de/view.php?id=48000 `width: 100%` on cloze text gaps ignored the configured HTML `size` and stretched every field to the full container. Use `width: auto` so the gap length applies again, and cap oversized fields with `max-width` to keep the scoring-modal overflow protection. Compile the change into `delos.css`.
thojou
force-pushed
the
fix-48000-Cloze-question-gap-too-long
branch
from
September 4, 2026 11:05
96e6ed2 to
107e799
Compare
Contributor
|
@thojou there is maybe a duplicated PR #12033 with belongs to the duplicated Mantis Issue: https://mantis.ilias.de/view.php?id=48210 |
thojou
approved these changes
Sep 4, 2026
thojou
left a comment
thojou
left a comment
Contributor
There was a problem hiding this comment.
Hey all,
Thanks @kunzeus for the PR. I reviewed the change and it fixes the regression.
I compiled the skin and integrated the compiled changes into this PR.
Also this seems to be an ILIAS 10 only issue, as the views and css rules have been changed for ILIAS 11.
If the CSS authorities are fine with these changes, we can proceed with merging.
Kind regards,
@thojou
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This fixes Mantis #48000 by restoring the configured Cloze text gap width instead of forcing all text inputs to width: 100%.
The change should keep large fields bounded so the overflow protection from Mantis #47413 remains intact.