-
Notifications
You must be signed in to change notification settings - Fork 40
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
Clean up the edit layout interface #373
Comments
I was going to post a similar screenshot suggestion for point 4 above, but then saw yours and yours looks better 😄 |
I am going to do some designs for this. |
👍 |
|
Any word from @Bojhan on this? Been hoping to see layouts polished and ready! |
Oops, sorry - I really forget about assigned issues in Github. I have some time scheduled this weekend to work on it! |
Thanks @Bojhan, appreciate it! |
What's left here after #891 ?? |
I think items 2 and 3 still need to be addressed, both relating to the title configuration. Let's split this out into a dedicated issue: #952 |
Based on feedback from #86
We need to clean up the edit layout interface, but none of the following issues were important enough to hold up the initial patch.
It's not clear how much of the page is the layout, and how much is just the rest of the admin page. There's a faint gray background on some of the page, but that's not enough. I wonder if adding a solid border around that area of the page might help. (I also think getting rid of stuff that's not part of the layout might also help - the title type selection needs to go!). Here's an attempt at adding a border around the "layout" area. (feedback please!)
The title type selection needs to go. All the other configurations are in modals, so I would love to see the "title" area of the page have a similar configure link, and a modal, where we can hide this title type business. (see screenshot below)
The page title needs to look more like a page title. As it is, it totally gets lost on the page. For starters, it needs to be a H1 instead of a H2. It should probably also be an "area" and not just a string of text, this can be accomplished with a border (and/or a background), like so:
The "add block" buttons are ambiguous as to which region they are adding the block into. Yes, there is a label next to each button, but we don't need to label things, we can just show them. Let's move the labels and buttons inside the region that will hold the block. And add some more spacing. Like so.
The text was updated successfully, but these errors were encountered: