wp-calypso icon indicating copy to clipboard operation
wp-calypso copied to clipboard

Themes global view - adapt theme preview for content frame.

Open Addison-Stavlo opened this issue 1 year ago • 5 comments

Currently when viewing the theme showcase at global view, clicking in to preview a theme shifts the interface away from global view.

We should find a way to keep the interface more consistent, such as showing the preview inside the global views content frame.

themes-goo

Note - that the Showcase exists as logged-in (single site, all sites) and logged-out. So any change should be tested in all cases

Addison-Stavlo avatar Jul 12 '24 16:07 Addison-Stavlo

This is a quick mockup, but perhaps we could use the collapsed sidebar and do something like this:

image

For the navigation we could use the Plugin breadcrumb.

javierarce avatar Aug 20 '24 12:08 javierarce

perhaps we could use the collapsed sidebar and do something like this:

Its not a bad idea but I think we want to be consistent with when we do and don't collapse the sidebar. The plugins browser in global for example shows individual plugins within the content frame with the sidebar open:

Screenshot 2024-08-20 at 4 34 11 PM

Originally writing this issue, I didn't expect needing a new design and we would literally just put the individual theme page inside the exact same content frame that the theme browser is in here:

Screenshot 2024-08-20 at 4 33 59 PM

If we wanted to use the collapsed sidebar, we should probably be consistent and do that with the plugins pages as well (not sure about others like domains?)

Or for now, just use the existing content frame and sidebar? It feels more like a bug that it breaks out of that to view the individual items.

Addison-Stavlo avatar Aug 20 '24 20:08 Addison-Stavlo

Or for now, just use the existing content frame and sidebar? It feels more like a bug that it breaks out of that to view the individual items.

Yes, sorry, let's do that. I originally thought we weren't displaying the regular left sidebar because we didn't want to shrink the columns, but I ran a quick test (which I should have done before publishing my first comment, sorry), and it's not too bad.

javierarce avatar Aug 21 '24 07:08 javierarce

Originally writing this issue, I didn't expect needing a new design and we would literally just put the individual theme page inside the exact same content frame that the theme browser is in

Let's do this @Addison-Stavlo. 👍

davemart-in avatar Aug 22 '24 12:08 davemart-in

I opened https://github.com/Automattic/wp-calypso/pull/93928 to fix. It'll need a couple of follow-ups like highlighting the menu item and changing the header, to be closer to the Plugins experience.

allilevine avatar Aug 27 '24 18:08 allilevine