-
Notifications
You must be signed in to change notification settings - Fork 127
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
Docs: move Fields to own pages #22
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for decap-www ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I am a bit concerned about the length of the sidebar at this point. Maybe I should look into collapsing them? I am also not sure that we are removing the name Widgets from the docs. I think these should be still called widgets because a Field and a Widget are different concepts. What about joining them into a subsection like this?
|
Yeah, I do get that concern. Perhaps collapsibles are needed now. I think it would honestly be an improvement in general..
Yeah, I agree. I'll make that change. |
I'm not happy with how this is looking at all. I think we should try to keep the buttons of the all the widgets in a list like it currently is: But refactor it so each widget is a first class page such as Image as opposed to an AJAXy box that is visually constrained.
With proposed change they are left aligned with normal page text. While the items under
|
I agree with all your points. Navigation: I think we need to put side menu items into accordions. That way other widget links will be visible all the time and we don't need the buttons. I agree with both of your other points. I can help with the sidebar and/or screenshots but I have some other tasks that I need to finish first, so it might take some patience. |
a608b23
to
b74884a
Compare
First pass attempt at #10 i'm not sure if what I did will satisfy Gatsby, but pushing trigger a build and investigate. This is a WIP