-
Notifications
You must be signed in to change notification settings - Fork 323
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
Replace Gridster with CSS flex grids #128
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds like a good plan. I've been using CSS flex grids with Vue and Vuetify, and it works well, and has good compatibility with browsers.
@YtvwlD could you rebase your branch and resolve conflicts? Then if that works similarly to Gridster we can merge it and release with 1.2.
I have rebased this onto current master. |
Thanks a lot for fixing it so quickly @YtvwlD ! I've just finished the retroactive changelog: https://github.com/Smashing/smashing/blob/master/CHANGELOG.md There are quite a few changes not released yet. I will cut 1.2.0 as-is first, and leave it in RubyGems for a while (1 month or more I guess?) to see if there's any issue reported. And in the meantime will review this one, check why Travis failed, and hopefully merge and prepare 1.3.0 with it. Thanks! |
@YtvwlD I've just rebased the PR, and added an extra commit that should fix the build on Travis. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a good PoC, but cannot be merged yet.
I think we just need to work on other options for drag n' drop. The multiple columns/rows could be achieved too I think, with just CSS (I think).
At the moment I get a scrollbar on Ffox and Chromium. Maybe because of the legend in one of the graphs. And some extra space in the widget with buzzwords.
Build failed only on 2.2.7, wihch is not a biggie now, as we can work around the other issues before worrying about this one. |
Posted on Gitter today, an alternative to Grister: https://github.com/hootsuite/grid |
there is also https://github.com/gridstack/gridstack.js |
Rebased, and removed the |
This is an attempt to upstream the changes I did to chaosdorf/dashpi in 2017. I don't know whether this can or should be merged but I figured it might be useful. (The commits should probably be squashed.)
What it does: Replaces Gridster with a few lines of CSS.
New features: widgets scale when the window size changes
Lost features: drag&drop, widgets more than two columns wide, widgets more than one row high
This requires adjustments to the dashboards, but affects just the sample project.