search

WordPress / wporg-theme-directory

15 stars 6 forks source link

Layout: Author #19

Closed tellyworth closed 4 months ago

tellyworth commented 1 year ago

Figma

(removed wrong screenshot to prevent confusion)

beafialho commented 1 year ago

Sorry, I must have left the wrong one in the Final page. Here's the correct mockup:

Figma

Themes - Author

Mobile Themes - Author

ryelle commented 1 year ago

@WordPress/meta-design This page seems to be the only one with "by [author]" on the theme card, and since it's the Author's archive page it seems redundant. Should we add that to the main theme grid too? or should it be gone from everywhere?

beafialho commented 1 year ago

I imagined the author name should only show up below the theme name in this page @ryelle, similar to what the site does now. However, removing it from everywhere also seems ok to me.

ryelle commented 1 year ago

huh. I wonder if that's because that page has no space for a "Themes by: AuthorName" header.

Since it would just link to the current page, let's leave it out. If we change our minds we can re-add it later.