Skip to content

remove unicode#11709

Merged
rotegras merged 3 commits into
ILIAS-eLearning:release_10from
rotegras:fix-47652-remove-spacing-breadcrumb
Jun 26, 2026
Merged

remove unicode#11709
rotegras merged 3 commits into
ILIAS-eLearning:release_10from
rotegras:fix-47652-remove-spacing-breadcrumb

Conversation

@rotegras

Copy link
Copy Markdown
Contributor

Remove unnecessary unicode character in breadcrumbs that causes unwanted spacing

@rotegras rotegras merged commit 50772c9 into ILIAS-eLearning:release_10 Jun 26, 2026
3 checks passed

@thibsy thibsy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @rotegras,

I was just made aware of this PR and your changes.

This is not how we handle things. As soon as files from other components are affected, where you do not have code-authority, you must wait for approval by the respective authorities. In this case, I would be the authority.

Since this is already integrated, please elaborate:

  • why you removed these characters? where does it lead to issues?
  • is this related to a mantis issue?
  • will these spaces be handled by CSS instead?
  • did you already cherry-pick these changes to other releases already?

Kind regards,
@thibsy (as UI coordinator)

@thibsy thibsy self-assigned this Jun 26, 2026
@thibsy thibsy added kitchen sink improvement css/html Pull requests that propose changes to CSS/SCSS or HTML files. labels Jun 26, 2026
@rotegras

Copy link
Copy Markdown
Contributor Author

Hi @thibsy, I am sorry for the inconvenience.

We looked at this issue with Bettina and agreed in the solution. I will wait next time for confirmation/revision.

Mantis Ticket is 47652. The issue appears only in ILIAS 10.

The unicode character is not visible but originates innecessary spacing before the breadcrumbs arrows.

Kind Regards

@rotegras (as CSS Authority)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

css/html Pull requests that propose changes to CSS/SCSS or HTML files. improvement kitchen sink

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants