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

Settings- and Info-Menue not scrollable #1668

Closed
dragricola opened this issue Jan 19, 2024 · 4 comments
Closed

Settings- and Info-Menue not scrollable #1668

dragricola opened this issue Jan 19, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@dragricola
Copy link

dragricola commented Jan 19, 2024

What happened?

At small displays like smartphone displays some menu items at the bottom side of the menu lists "Settings" and "Info" are not accessible because they are outside of the display area and the menu window is not scrollable.

To Reproduce Bug

This problem only appears on small displays like smartphones under the menu items Settings (Einstellungen) and Info.

Expected Behavior

It would be fine, when configuration and informations of openDTU can be also done with a smartphone.

Install Method

Pre-Compiled binary from GitHub

What git-hash/version of OpenDTU?

v24.1.18 and many versions before

Relevant log/trace output

No response

Anything else?

Screenshot_20240119_152824_Firefox 1

This screenshot shows an example, where the menu points below DTU-Settings are not accessible.

@dragricola dragricola added the bug Something isn't working label Jan 19, 2024
@nkappler
Copy link
Contributor

nkappler commented Jan 27, 2024

I've just investigated this a little bit, the fix is not that trivial, since there's another case where scrolling should appear:
grafik

I've tried a css only solution but since the same DOM elements are used for both layouts, it's tricky...

nkappler added a commit to nkappler/OpenDTU that referenced this issue Jan 27, 2024
@nkappler
Copy link
Contributor

ok I've got a solution now...
That was quite an interesting challenge, I've learned a bit more about vue.js, thanks 🥳😂

@dragricola
Copy link
Author

ok I've got a solution now...

Thank you! Your solution works very fine!

nkappler added a commit to nkappler/OpenDTU that referenced this issue Feb 8, 2024
Copy link

github-actions bot commented Apr 1, 2024

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants