Skip to content

Conversation

@math-GH
Copy link
Contributor

@math-GH math-GH commented Sep 1, 2025

Before:
grafik

1: menu is not right hand side aligned
2: rounded corner (Origine theme specific)

After:
grafik

Menu is aligned on the right hand side
Sharp corner

Changes proposed in this pull request:

  • CSS

How to test the feature manually:

  1. small screen for mobile view
  2. open config menu right top corner

Negative check:
other dropdown menus should not be touched (f.e. User Queries menu, search, article labels menu)

@math-GH math-GH added this to the 1.28.0 milestone Sep 1, 2025
@math-GH math-GH added UI 🎨 User Interfaces Theme 🖌 labels Sep 1, 2025
height: 100vh;
top: 0;
right: 0;
right: 0 !important;
Copy link
Member

Choose a reason for hiding this comment

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

I would be happier if we could avoid all the !important but that can wait

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, me too

@Alkarex Alkarex merged commit 1b2293a into FreshRSS:edge Sep 1, 2025
1 check passed
@math-GH math-GH deleted the fix/config-dropdown-menu-mobile-view branch September 2, 2025 12:16
@Alkarex Alkarex modified the milestones: 1.28.0, 1.27.1 Sep 2, 2025
@Inverle
Copy link
Member

Inverle commented Sep 3, 2025

Wouldn't it be better to align the menu in the middle or full screen?

@math-GH
Copy link
Contributor Author

math-GH commented Sep 3, 2025

Wouldn't it be better to align the menu in the middle or full screen?

The idea behind is:
The config menu button opens the menu right side aligned.
The "folder" icon button opens the main navigation left side aligned

@Frenzie
Copy link
Member

Frenzie commented Sep 3, 2025

It made sense to me that way.

@Inverle
Copy link
Member

Inverle commented Sep 3, 2025

The idea behind is:
The config menu button opens the menu right side aligned.

I just got too used to it being on the left side that I got kinda confused, but I like it now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Theme 🖌 UI 🎨 User Interfaces

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants