-
Notifications
You must be signed in to change notification settings - Fork 31
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
feat: Responsive web frontend #429
base: main
Are you sure you want to change the base?
Conversation
…late for desktop Navigation rail
Hi @JM-Rib, I did notice one issue: the frontend tests are failing due to the following warning:
It should be sufficient to remove the import from the import 'package:plant_it/commons.dart'; |
Hi ! Thanks for the feedback, it's fixed now! |
Thanks for the work on this. I have a question to better understand the implementation effort involved. Would it be feasible to add an option in the app settings to enable/disable this new UI? Perhaps we could place it below the "Server" section in a new "App" section, with a setting called "Wide UI." This would allow users to choose between the mobile version and the new UI, depending on their preference. Looking forward to your thoughts on this! |
Hi, that would be a great implementation idea! I'll look into it and keep you up to date once I come up with something. I can definitely see the specifics of it and how this would work out. |
Responsive web frontend
This pull requests adapts the interface to a desktop sized screen in a responsive manner.
What's new?
Why is it important?
How to Use?
Screenshots