-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Refreshed Fullpost Design Idea: New "visit site" link #8791
Comments
This would link to the home page of their site instead of the post itself, correct? I think that'd help, although its location may be easy to miss on wider screens, but we can give it a shot. We should also add an event to this to see how often people click it. How do you envision it on mobile? This may feel cramped, but maybe just the external link icon next to the Like icon? Another idea is an external link icon right after the post title. |
@jancavan No - this would link to the permalink page. The same URL linked from the title and datestamp now. (I know "VISIT SITE" makes it sound like the homepage, but I needed a word to make it clear that what you'd be visiting is the parent site.) And: I love the idea of tracking clicks on this. As for mobile, you think there's space to put it on the right side in the top bar, to keep it on the same line as "BACK"? Maybe shorten it to "VISIT"? Or maybe just use the icon in the top right for mobile? |
That's better! Following may overlap with Comments if we consider i18n. Although I forget if full-post test is English only? But what about dropping Follow/Following labels on mobile? |
We could to that, too, though I wouldn't want to unless it's absolutely necessary. It's important that is say "Follow" when you're not following to encourage it. |
Fullpost test is all users, not just English, FWIW. |
|
Just checked, it's actually English only. |
* Reader: Add in better post permalink tracking. Fixes #8791 * bind recordDateClick to the instance so this.props resolves * differentiate full post and card timestamp clicks
* Reader: Add in better post permalink tracking. Fixes #8791 * bind recordDateClick to the instance so this.props resolves * differentiate full post and card timestamp clicks
Currently, on the Refreshed Fullpost, the post title and date stamp link to the original post.
I'd like to suggest adding a new link to the original post at the top, right.
Design details: the link matches the same type treatment as the "BACK" link, and is on the same baseline. The color of the icon matches the color of the back arrow. The idea is to mirror that functionality, so the left side is back to the stream, and the right side is forward to the post.
Why do this? The links on the title and datestamp are not obvious. This change would make it obvious how to get to the parent blog if the user wants to see the post in context. Now that we're showing the full text on fullpost, some are complaining. Giving users a clear way to get to the parent blog might assuage a bit of that concern. After all, we want to give people an obvious way to get to the parent blog if that's what they want to do.
Thoughts @jancavan?
The text was updated successfully, but these errors were encountered: