Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fixed: hover over title while having the navigation buttons in mobile view #6486

Merged
merged 1 commit into from
May 23, 2024
Merged

fixed: hover over title while having the navigation buttons in mobile view #6486

merged 1 commit into from
May 23, 2024

Conversation

math-GH
Copy link
Contributor

@math-GH math-GH commented May 23, 2024 •
edited
Loading

I guess it is a regression from #6444

before:
grafik

After:
grafik

Changes proposed in this pull request:

  • CSS: z-index

How to test the feature manually:

  1. normal view in a small screen (mobile view)
  2. hover over a article line that is partially hidden behind the navigation buttons

Pull request checklist:

  • clear commit messages
  • code manually tested

@math-GH math-GH added the UI 🎨 User Interfaces label May 23, 2024
@math-GH math-GH added this to the 1.24.1 milestone May 23, 2024
@Alkarex Alkarex merged commit 7f61a44 into FreshRSS:edge May 23, 2024
2 checks passed
@math-GH math-GH deleted the fix-hover-and-nav-buttons branch May 23, 2024 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
UI 🎨 User Interfaces
Projects
None yet
Milestone
1.24.1
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@math-GH @Alkarex