Profiles filter context missing #1355
Conversation
|
@JakePaustianISU is attempting to deploy a commit to the Mastodon Team on Vercel. A member of the Team first needs to authorize it. |
vmstan
left a comment
vmstan
left a comment
There was a problem hiding this comment.
We probably also need to update the screenshots on this page since elements have changed.
andypiper
left a comment
andypiper
left a comment
There was a problem hiding this comment.
Looks fine, but as stated in the PR, this page would benefit from updated images as well.
| * Notifications = matching notifications will not be shown | ||
| * Public timelines = matching statuses will not appear in local/federated timelines | ||
| * Conversations = matching statuses will be hidden in threads and detailed views | ||
| * Profiles = matching profiles will be hidden |
There was a problem hiding this comment.
matching profiles will be hidden -> matching statuses will be hidden in profile views?
There was a problem hiding this comment.
that would be a better way of phrasing it.
andypiper
left a comment
andypiper
left a comment
There was a problem hiding this comment.
suggest updating the language per comments in the review.
| * Notifications = matching notifications will not be shown | ||
| * Public timelines = matching statuses will not appear in local/federated timelines | ||
| * Conversations = matching statuses will be hidden in threads and detailed views | ||
| * Profiles = matching profiles will be hidden |
There was a problem hiding this comment.
that would be a better way of phrasing it.
|
This pull request has merge conflicts that must be resolved before it can be merged. |
4 participants
Trys to address issue #1332