-
-
Notifications
You must be signed in to change notification settings - Fork 37
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
Responsive toolbar layout #274
Comments
To be honest, it would be nice if there was a layout property and its choices were: scroll, wrap, moreMenu, etc... Since I stared my Flutter app a little over a year ago and learning along the way, I am learning to be adaptive and creative in ways I was never forced before while programming in Windows for 30+ years. Flutter's cross-platform ability is very unrestrictive and thinking outside the box as opposed to being constrained by it like a lot of other frameworks is refreshing. So allowing more options would be better. Thanks for everyone's work! All the best, Aggie85 |
Hello, I'm not a developer of Feather, just a curious tourist doing sand castles, so the following is only my opinion: As a developer yourself, you already understand that the multiplicity of options impacts the maintainability. I think that the developers times can be better used to missing core components before creating a more complex front-end. Also, at this stage of development of Fleather, front end can be easily impacted by profound changes in the back-end. I'm not saying here to not do something. I personally like your idea of options. There is already a FleatherToolbar.basic() factory, there can be others like FleatherToolbar.scroll(), etc... Actually, you made me thinking on the problem and I see different avenues to choose:
What do you think ? If you want to share more of your point of view, a little debate is occurring in this PR: #277 |
Is your feature request related to a problem? Please describe.
Take your example app, resize it to be smaller than the width of the toolbar. Right-most buttons are hidden.
Describe the solution you'd like
See below, I prefer the first solution.
Describe alternatives you've considered
Different solutions:
The text was updated successfully, but these errors were encountered: