-
Notifications
You must be signed in to change notification settings - Fork 5
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
Showcase v2 Feedback #234
Comments
Good stuff here. Marko is a bit busy at the moment so I'll respond to the best of my ability. In general, agree with most of this, so the stuff I didn't comment on/add context to below, would be good to just run with.
Conceptually it's the feature version of this box; At the same time, it should remain minimal. We could look to adding icons to denote the category and URL, and in this case, "Meta Newsroom" is the title. The fact that Facebook rebranded to Meta is a point of confusion that's harder for us to fix.
I think Kelly brought this up, and there should be a separate issue to just add those titles. Let's do that.
Perhaps there's a better example of the general feedback about descriptive introductions, but the feedback doesn't feel entirely valid for the Browse page, as there are both breadcrumbs and a clear title on the page. If we need to add a short paragraph below, that won't be the end of the world, I could just use some clarification on the ask.
Agree.
I feel like this issue captures the problem, which to be clear is an important one to solve. Home should link to the landing page for Showcase, but it should be de-emphasized and as was suggested in the design on that issue, probably be a unified bar, because Showcase will also have a top level item. CC: @fcoveram — we've discussed here a "compact" mode for the two navigation bars that merge them into one for these cases.
Assuming this is referring to these: I don't mind them. I also think we can remove them if need be. No strong opinion here. |
I brought that up in the initial review. Here is context. |
@jonoalderson would you be able to elaborate on this one? The search pages do appear to use the |
See the canonical URL tag. Actually, given that the page is noindex'd, the canonical URL tag should be removed entirely. |
I don't think it makes sense to prioritize visual consistency between the banner and the single post, as they serve different purposes. It may well make sense to use a completely different visual treatment, have different content, etc, for people who're seeing the Showcase homepage (and saying "what on earth am I looking at?"), vs those who navigate to or land directly on an individual Showcase item where the context is very different. |
I think we're close, but I'm nervous that we're really not hitting the mark on "does the homepage of this thing make sense, clearly convey the right message, and help users get to the right place" with that header area. We get one chance to launch this and make a song and dance. Instead of spending weeks measuring (unsurprisingly few?) interactions with what we know is a confusing interface, I'd vote that we spend just a tiny bit more time and reinvent this key component before shipping it. That transformation to a statement is much more intuitive, and I don't think there's a huge amount of extra work needed to do something more meaningful in that space than the box of links. |
I agree that this UI could be slightly more custom. And I like where your exploration is going @ndiego I'm not entirely sure if we should hide the url because some instances might be a sub-site of the mother company (let's say Wirecutter within NYT) which would get the titles too long? And perhaps the url is more specific. Even if it's less prominent. Plus it gives a way to open in new tab the real site. I'm not sure about the main CTA above (if it's intended to direct to the real site). If it's meant to go to the site detail page, is the link/button label clear? is it Learn more? Not a strong opinion, but because I don't see the url, it feels I'm going to the site itself. |
Copied over from WP Slack: https://wordpress.slack.com/archives/C04U953K77A/p1696925380920099?thread_ts=1696525287.393319&cid=C04U953K77A
<title>
s are inconsistent about whether they include (or end in) "WordPress Showcase"; can we define and use a consistent pattern throughout which includes this?Home > Showcase >
etc? Or if that's contentious, just replace 'Home' with 'Showcase' in the breadcrumbs.noreferrer
on links to sites?The text was updated successfully, but these errors were encountered: