Skip to content

Compact left menu#3389

Merged
ildyria merged 1 commit intomasterfrom
left-menu-compact
Jun 1, 2025
Merged

Compact left menu#3389
ildyria merged 1 commit intomasterfrom
left-menu-compact

Conversation

@ildyria
Copy link
Copy Markdown
Member

@ildyria ildyria commented May 30, 2025

Before After
image image

@ildyria ildyria requested a review from a team as a code owner May 30, 2025 15:32
@ildyria ildyria added the Review: easy Easy review expected: probably just need a quick to go through. label May 30, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented May 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.99%. Comparing base (22676e7) to head (c53d2a9).
Report is 8 commits behind head on master.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ildyria ildyria added the alpha-ready Branch is available in alpha label May 31, 2025
Copy link
Copy Markdown

@RonnieTaz RonnieTaz left a comment

Choose a reason for hiding this comment

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

This looks well with me, but in future, could we reimplement the relaxed menu and make as an option of sorts so user can choose if they want compact or relaxed menu

Copy link
Copy Markdown

@RonnieTaz RonnieTaz left a comment

Choose a reason for hiding this comment

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

Looks good to me

@ildyria
Copy link
Copy Markdown
Member Author

ildyria commented Jun 1, 2025

This looks well with me, but in future, could we reimplement the relaxed menu and make as an option of sorts so user can choose if they want compact or relaxed menu

It is actually just changing the padding, not much implementation needed. the thing is that the "Lychee" part is taking quite some space and I am planning to move it else where.
And the rest is the Admin part which "normal users" registered in Lychee do not see :)

@ildyria ildyria merged commit 0bca5e7 into master Jun 1, 2025
35 checks passed
@ildyria ildyria deleted the left-menu-compact branch June 1, 2025 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

alpha-ready Branch is available in alpha Review: easy Easy review expected: probably just need a quick to go through.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants