Skip to content

fix(ui): replace obsolete 'ti-external-link-alt' with 'ti-external-link' - #25407

Merged
trasher merged 1 commit into
glpi-project:11.0/bugfixesfrom
aalexanderdev:fix/tabler-external-link-icon
Sep 7, 2026
Merged

fix(ui): replace obsolete 'ti-external-link-alt' with 'ti-external-link'#25407
trasher merged 1 commit into
glpi-project:11.0/bugfixesfrom
aalexanderdev:fix/tabler-external-link-icon

Conversation

@aalexanderdev

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

  • I have read the CONTRIBUTING document.
  • I have performed a self-review of my code.

Description

In GLPI 11 (after migrating to Tabler Icons), the class ti-external-link-alt was accidentally carried over from legacy FontAwesome (fa-external-link-alt).

Since Tabler Icons only defines .ti-external-link (without any -alt suffix), the <i> element matches no CSS rules, rendering an invisible 0px × 0px element.

In the Plugins table (front/plugin.php), because the link text is also hidden via <span class="sr-only">, the plugin website link is completely invisible to users.

Changes

  • Replace ti-external-link-alt with ti-external-link in src/Plugin.php (plugin homepage column).
  • Replace ti-external-link-alt with ti-external-link in templates/install/accept_license.html.twig (installer license link).

@trasher
trasher merged commit 132f797 into glpi-project:11.0/bugfixes Sep 7, 2026
13 checks passed
@cedric-anne cedric-anne added this to the 11.0.9 milestone Sep 7, 2026
@cedric-anne cedric-anne added the ui label Sep 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants