-
Notifications
You must be signed in to change notification settings - Fork 30
Unship panel layout #301
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
Unship panel layout #301
Conversation
✅ Deploy Preview for lumen-notes ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for lumen-storybook ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I was about to open an issue because I thought I broke something. I like the multi-column. |
@NetOpWibby That's great feedback. What do you miss about the multi column layout? |
No wasted space! And, I got to see multiple notes at once. |
@colebemis The screenshot in the README should probably be updated too. |
This PR removes the panel system in favor of a simple one-column layout. In practice, the complexity of the panel layout outweighed the benefits. I'm going to ship this one-column layout to see how it feels in day-to-day use.