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

Hide button when scrolling in mobile component #192

Open
mcicolella opened this issue Oct 14, 2018 · 12 comments
Open

Hide button when scrolling in mobile component #192

mcicolella opened this issue Oct 14, 2018 · 12 comments

Comments

@mcicolella
Copy link
Collaborator

As you can see in the attached image, the button (bottom right corner) overlaps some elements.

screenshot_2018-10-14-18-11-28

It'd better to hide it when scrolling down and show it again when scrolling up.

P.D. any other good alternative?
Code ref. https://github.com/freedomotic/fd-vue-webapp/blob/master/src/components/mobile/mobile-pages/PluginsPanel.vue

@amenak77
Copy link

amenak77 commented Oct 15, 2018

If object's list lenght is lower then or equal to screen's lenght, the button could cover an element anyway and the list shouldn't be "scrollable". I guess we should think to something else

@mcicolella
Copy link
Collaborator Author

@amenak77 what's your solution?

@amenak77
Copy link

@mcicolella maybe changing the button's position

@amenak77
Copy link

or commutate the button with another GUI element less "intusive"

@amenak77
Copy link

In my opinion, first choise should be more standard

@romanofranz
Copy link

What about changing the floating button to a menu entry in the app bar menu?

@amenak77
Copy link

@romanofranz choise numer two; not bad
@mcicolella what do you think about this?

@amenak77
Copy link

@romanofranz do you want to take this issue?

@romanofranz
Copy link

@amenak77 Unfortunately I'm not a Vue developer, I was just suggesting my idea because I spot the question on Twitter :(

@mcdead154
Copy link

Hi, i would like to take this issue as my first contribution, if it is still relevant.

@mcicolella
Copy link
Collaborator Author

@mcdead154 ok

@anilakodukula
Copy link

is this still an issue? The link above to the code does not work.

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

No branches or pull requests

5 participants