-
-
Notifications
You must be signed in to change notification settings - Fork 576
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
fix(ui): move account lock and bot icons next to name in timeline #2772
base: main
Are you sure you want to change the base?
Conversation
Run & review this pull request in StackBlitz Codeflow. |
✅ Deploy Preview for elk-docs canceled.
|
✅ Deploy Preview for elk-zone ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
48e0676
to
6fec751
Compare
To me it's more about the position of the icon. I think if the lock is for the account it should be visible & closer to the name/profile picture. Then maybe we can also show consistently in different profile views? I wonder if we could have two rows in the status card similar as in the profile page... the name + icons (at top) and username (bottom) |
…untBotIndicator to next to account name
Sorry for the late response! Right, it's better to have adjusted the icon position instead of reducing information by deleting this. I think changing to two rows is a good option worth exploring. But that will change all post height and layout in a timeline, so I'm not sure how that UI change affects the current design. I'd like to create a separate PR for that change. Here's new position for the lock icon. I also moved the bot icon as they should be in the same group (attributes of the author's account). |
6fec751
to
bdb3c30
Compare
✅ Deploy Preview for elk-docs canceled.
|
fix #2492
As explained in the original issue, showing the account lock 🔒 icon on the status card is quite confusing:
I think this decision in the official Mastodon UI is reasonable because the meaning of 🔒 icon on Mastodon differs from that on Twitter.
On Twitter, 🔒 icon for the account has two meanings at the same time: the account requires to follow requests, and the visibility of all the posts is limited to "Only follower". So showing 🔒 icon on the status makes sense.
On the other hand, on Mastodon, the visibility of posts and account following limitations are separated. And the locked account still can publish publicly visible posts so 🔒 icon only indicates that the following approval is required when someone is following.
Mastodon already has the icon for visibility on the post (such as 🌎 and 🔒) so it should be enough to show 🔒 icon only on the account page (or other account-specific component) where users need to know whether the account requires the following request or not.
Although Mastodon UI doesn't show the bot icon too, I think we can keep it because the bot icon can show both meanings: the account is automated and the post itself is programmatically published by the bot.
After
The status detail page still shows the lock indicator as the visibility indicator is clearly separated in different places without confusion.
Another example with the lock icon: