-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add comments pattern #487
Comments
These patterns are feeling kind of cramped in longer threads. I also think we may have flipped the hierarchy the opposite of what it should be, where author/date might be more important up top. I think we were designing these as if they were tweet-length, but since then we've had some pretty long comments where it's unclear who is talking till you've finished reading. |
The last week or two has been kind of interrupty for me, before the holiday I thought I'd share some of my progress. Here's a chunk of the comment thread for one of our articles on our live site on a narrow screen: I like how clean it is and how the threading is straightforward. I dislike that the name and info is below the comment… it nicely consolidates meta information to one line, but when comments get longer, it omits the context of who is actually speaking. I also think it ends up feeling more and more cluttered the longer the thread gets, as everything is quite condensed. I've been experimenting with a layout more like this (same content but with random placeholder avatars): Things I like:
Thing I'm not quite satisfied with:
Anyway! There's a progress update. |
Here is the PR for the prototypes for this design: #1230 |
Create a pattern for comments, comments can be found on article pages.
The text was updated successfully, but these errors were encountered: