WordPress / wporg-theme-directory

15 stars 6 forks source link

Improve preview #64

Closed jasmussen closed 3 months ago

jasmussen commented 3 months ago

Image

Overall looks good. The sidebar has been made useful with patterns and style variations. A few notes:

Finally, the download button is sticky at the bottom so it’s easy to see, that’s good. But it’s a long way to tab if you’re just using the keyboard, having to go through style variations and patterns first. Maybe we can move it to the top, right after the description? A bit like this sketch:

Image

ryelle commented 3 months ago

Just FYI, I've moved this back to pending discussion & added the Needs Design Feedback label since there are still unanswered design questions here.

Should we move the Wide, Medium, and Narrow buttons to the top, to match their placement for pattern resizing?

We don't have the same resizing issue that we initially had with the toggles on Patterns, and this UI is very different from that one IMO. Plus, this selector is at the bottom of the sidebar on the old theme, so I would lean to leaving it at the bottom.

Style variations and Patterns should both have the same heading style.

Which one?

Perhaps we should make the Download button slightly smaller

Does the "small" variation work?

Screenshot 2024-05-13 at 11 15 13 AM

Maybe we can move it to the top, right after the description?

Sure, and styles/patterns would appear after that? Should there be any separator or extra space under the zip name?

jasmussen commented 3 months ago

We don't have the same resizing issue that we initially had with the toggles on Patterns, and this UI is very different from that one IMO. Plus, this selector is at the bottom of the sidebar on the old theme, so I would lean to leaving it at the bottom.

We can go with that, sure, can always revisit.

Style variations and Patterns should both have the same heading style.

Which one?

I opened a lot of issues yesterday, and thanks for diving in, I should've been more clear.

This EB Garamond heading style works well: Screenshot 2024-05-14 at 09 12 56

Works better than the paragraph style here:

Screenshot 2024-05-14 at 09 13 00

So if it could be just a heading for style variations in similar style to Patterns, that would work well.

Does the "small" variation work?

Yes.

Sure, and styles/patterns would appear after that? Should there be any separator or extra space under the zip name?

Most likely. Here's a quick inspector sketch of how it could look, all of it coming together: Screenshot 2024-05-14 at 09 18 43

Note here the font size of the Style variations and Patterns headings is the H5 font size of 26px. Let me know if that works for you.