Skip to content

UI tweaks#1094

Merged
sgiehl merged 3 commits into5.x-devfrom
ui-tweaks
Apr 24, 2026
Merged

UI tweaks#1094
sgiehl merged 3 commits into5.x-devfrom
ui-tweaks

Conversation

@chippison
Copy link
Copy Markdown
Contributor

Description

This will be the PR we will use to merge in our 3 UI tweaks
(Left Menu, Widgets/Cards and Background, Main Selectors)
Currently this only has

  • Left Menu
  • Widgets/Cards and Background

Issue No

DEV-19958
DEV-19960

Checklist

  • [✔] Tested locally or on demo2/demo3?
  • [NA] New test case added/updated?
  • [NA] Are all newly added texts included via translation?
  • [NA] Are text sanitized properly? (Eg use of v-text v/s v-html for vue)
  • [NA] Version bumped?
  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules
  • [NA] Documentation updated?

@chippison chippison force-pushed the ui-tweaks branch 9 times, most recently from 83e9b1a to bd75e60 Compare April 23, 2026 11:48
@chippison chippison requested a review from a team April 24, 2026 07:35
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.

⚠️ The first row do not have the same background. Perhaps a problem of missing hover effect or a regression.

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.

⚠️ Same point of attention here: the color of the first row is not consistent

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.

⚠️ Here too

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.

⚠️ here too, but in the other direction

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.

⚠️ here again

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.

⚠️ here again

@sgiehl sgiehl merged commit 7839b40 into 5.x-dev Apr 24, 2026
13 checks passed
@sgiehl sgiehl deleted the ui-tweaks branch April 24, 2026 15:56
This was referenced Apr 29, 2026
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.

3 participants