WordPress / wporg-theme-directory

15 stars 6 forks source link

Author archive page should have visible heading #78

Closed fcoveram closed 2 months ago

fcoveram commented 3 months ago

Is your feature request related to a problem? Please describe. Theme creators' page looks very similar to the search results one and it's not very clear that is listing themes from a creator.

Screenshot of theme creator's index page

Describe the solution you'd like Add a text saying "by [creator-name]" next to the results label on the right side of the search input. For instance, in this case it would say "27 themes by Creta Themes".

If this solution is too subtle, we could consider adding a heading like the h1 of theme details page saying "[creator-name]'s themes" or something similar

ryelle commented 3 months ago

The page already has a hidden heading for accessibility (I mentioned in this plugns thread how that's kind of strange, that only the screen-reader users get that benefit); I could make that visible? Here's an attempt with some style:

Screenshot 2024-05-23 at 12 48 23 PM

That could be just on authors, or I could make that visible everywhere:

Screenshot 2024-05-23 at 12 48 36 PM

Or we could bring in breadcrumbs like patterns:

Screenshot 2024-05-23 at 12 48 58 PM

Finally, just noting that this is visible on scroll:

Screenshot 2024-05-23 at 12 49 28 PM

What do you think?

fcoveram commented 3 months ago

Making the heading visible on Authors is enough. I didn't notice the label in local nav when scrolling down, so showing the heading in both local nav and below search and filter bar looks good.