Skip to content

style(AppsMenu): adjust spacing - #29

Merged
bromiesTM merged 1 commit into
mainfrom
kh/dev/theme-adjustment
Oct 16, 2025
Merged

style(AppsMenu): adjust spacing#29
bromiesTM merged 1 commit into
mainfrom
kh/dev/theme-adjustment

Conversation

@bromiesTM

Copy link
Copy Markdown
Contributor

No description provided.

@bromiesTM
bromiesTM requested a review from Copilot October 14, 2025 12:23

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adjusts the spacing and sizing configuration for the AppsMenu component to improve the visual layout and user experience.

  • Reduced grid image size from 53px to 35px for more compact display
  • Added explicit width and height properties to the trigger icon (24px)
  • Increased menu dimensions (width from 320px to 350px, max-height from 400px to 600px)
  • Replaced CSS variable padding with hardcoded values (30px 15px 30px 30px)
  • Removed max-height calculation for the grid to simplify layout
Comments suppressed due to low confidence (1)

src/lib/view/AppsMenu.vue:1

  • The removal of the max-height calculation may cause layout issues. The calculation was dynamically sizing the grid based on the number of visible rows and spacing variables. Without this constraint, the grid may grow beyond intended bounds or create inconsistent layouts.
<!--

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/lib/view/AppsMenu.vue
Comment thread src/lib/view/AppsMenu.vue Outdated
Signed-off-by: Kai Henseler <kai.henseler@strato.de>
@bromiesTM
bromiesTM force-pushed the kh/dev/theme-adjustment branch from 3fc8b6f to 0b3286a Compare October 16, 2025 07:58
@bromiesTM
bromiesTM merged commit 0b6fe0d into main Oct 16, 2025
18 checks passed
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