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.

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

Change layout for the share options on mobile #1645

Merged
merged 2 commits into from
Sep 28, 2017

Conversation

aledeg
Copy link
Member

@aledeg aledeg commented Sep 27, 2017

Before, it was impossible to click on links to share on small screens.
Now, all dropdowns are opened at the bottom of the screen and stays there.

Linked to #1506

Before, it was impossible to click on links to share on small screens.
Now, all dropdowns are opened at the bottom of the screen and stays there.

Linked to FreshRSS#1506
@Frenzie
Copy link
Member

Frenzie commented Sep 27, 2017

Where is this dropdown menu? Whenever I see !important my spidey senses start tingling. :-P

@aledeg
Copy link
Member Author

aledeg commented Sep 27, 2017

@Frenzie I know :)
The dropdown menu are the one that you can see when you click on the share button and on the user queries. I used !important because I want to make sure that there is no radius on that dropdown since I display it at the bottom of the screen and it uses the whole space. The round corner was quite disgraceful.
Feel free to come with a better solution.

@Alkarex Alkarex added the UI 🎨 User Interfaces label Sep 27, 2017
@Alkarex Alkarex added this to the 1.7.1 milestone Sep 27, 2017
@Alkarex
Copy link
Member

Alkarex commented Sep 27, 2017

Would it be possible to find a more qualified rule, hence avoiding the need of !important ?

@Frenzie
Copy link
Member

Frenzie commented Sep 27, 2017

I don't really want to figure out where this sharing thing is right now[1] but for the user query button you could increase specificity by using something like .dropdown .dropdown-menu.

[1] It looks like my settings say sharing in the bottom line but I suppose I'd have to set up some sharing for it to display.

@aledeg
Copy link
Member Author

aledeg commented Sep 27, 2017

@Frenzie @Alkarex Done. I removed the !important

@Alkarex Alkarex merged commit 47abdd0 into FreshRSS:dev Sep 28, 2017
Alkarex added a commit that referenced this pull request Sep 28, 2017
@aledeg aledeg deleted the share-on-mobile branch September 29, 2017 04:58
javerous pushed a commit to javerous/FreshRSS that referenced this pull request Jan 20, 2020
* Change layout for the share options on mobile

Before, it was impossible to click on links to share on small screens.
Now, all dropdowns are opened at the bottom of the screen and stays there.

Linked to FreshRSS#1506

* Change rule for border-radius
javerous pushed a commit to javerous/FreshRSS that referenced this pull request Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI 🎨 User Interfaces
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants