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

delete the no-mobile css class #4128

Merged
merged 1 commit into from
Jan 6, 2022
Merged

delete the no-mobile css class #4128

merged 1 commit into from
Jan 6, 2022

Conversation

math-GH
Copy link
Contributor

@math-GH math-GH commented Jan 6, 2022

Partially #4127

Changes proposed in this pull request:

  • show the full dropdown menu of user query menu
  • deleted the no-mobile CSS class

How to test the feature manually:

  1. open the dropdown of the user query menu
  2. see the cog icon
  3. see the bookmark menu entry

Pull request checklist:

  • clear commit messages
  • code manually tested

@math-GH math-GH added the UI 🎨 User Interfaces label Jan 6, 2022
@math-GH math-GH added this to the 1.19.2 milestone Jan 6, 2022
@Alkarex Alkarex merged commit f8c5df2 into FreshRSS:edge Jan 6, 2022
@math-GH math-GH deleted the full-user-query-menu-in-mobile-view branch January 8, 2022 08:47
@math-GH
Copy link
Contributor Author

math-GH commented Jan 11, 2022

Just for the history books: it was my 100th commit into edge (=PR)

@Alkarex
Copy link
Member

Alkarex commented Jan 11, 2022

Well done @math-GH , and appreciated !

@Alkarex Alkarex mentioned this pull request Jan 29, 2022
Closed
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.19.2
Development

Successfully merging this pull request may close these issues.
2 participants
@math-GH @Alkarex