Skip to content

fix(service_catalog): limit card height in service catalog#23880

Open
MyvTsv wants to merge 2 commits intoglpi-project:11.0/bugfixesfrom
MyvTsv:ticket43331
Open

fix(service_catalog): limit card height in service catalog#23880
MyvTsv wants to merge 2 commits intoglpi-project:11.0/bugfixesfrom
MyvTsv:ticket43331

Conversation

@MyvTsv
Copy link
Copy Markdown
Contributor

@MyvTsv MyvTsv commented Apr 15, 2026

  • I have read the CONTRIBUTING document.
  • I have performed a self-review of my code.
  • I have added tests that prove my fix is effective or that my feature works.
  • This change requires a documentation update.

Description

  • It fixes !43331
  • This fix limits the height of the cards to prevent the service catalog from becoming unreadable

Screenshots (if appropriate):

After:
image

Before:
image

@MyvTsv MyvTsv self-assigned this Apr 15, 2026
@MyvTsv MyvTsv requested a review from Rom1-B April 15, 2026 09:15
Comment on lines +69 to +70
class="text-secondary remove-last-tinymce-margin overflow-hidden"
style="max-height: 300px"
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.

Is 300px still too much? (based on your screenshot examples)

Additionally, the full description can be displayed when you hover over it with the mouse?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I can reduce the pixel size. However, I don't think displaying all the text is a good idea in terms of usability when the user hovers over a map, because the interface would flicker a lot.

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 this pull request may close these issues.

2 participants