Skip to content
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

Change label of status visibility to "Only people I mention" in web UI #18146

Merged
merged 1 commit into from
Apr 28, 2022

Conversation

Gargron
Copy link
Member

@Gargron Gargron commented Apr 28, 2022

To avoid confusion with proper, end-to-end encrypted direct messaging functionality that's planned for the future

@Gargron Gargron force-pushed the fix-remove-conversations-column branch from 59c5bfc to 225a4db Compare April 28, 2022 05:58
@ClearlyClaire
Copy link
Contributor

The PR not only renames it but also removes a lot of menu items and remove the DM timeline altogether… While I understand some people have incorrect expectations because of how this is named, I think removing that much functionality isn't a good thing.

@Gargron Gargron force-pushed the fix-remove-conversations-column branch from 225a4db to f5f96a9 Compare April 28, 2022 19:53
@@ -10,7 +10,7 @@ import { connectDirectStream } from '../../actions/streaming';
import ConversationsListContainer from './containers/conversations_list_container';

const messages = defineMessages({
title: { id: 'column.direct', defaultMessage: 'Direct messages' },
title: { id: 'column.conversations', defaultMessage: 'Conversations' },
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about “personal message”? Still convey a similar idea but more reminiscent of BBS and similar communities smaller than Twitter.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PMs and DMs are often used interchangeably and I would like to avoid any confusion

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Honestly I fail to see how “Personal Message” may convey an idea of encryption, like I said it's pretty much reminiscent of BBS and other small-scale forums where you have unencrypted PMs with admins you may personally know.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My problem with “Conversations” is that we already use that term for threads (e.g. “Mute conversation”, and what I wanted to do for the improved filtering system)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I give up, Conversations is fine.

app/javascript/mastodon/features/direct_timeline/index.js Outdated Show resolved Hide resolved
app/javascript/mastodon/features/getting_started/index.js Outdated Show resolved Hide resolved
app/javascript/mastodon/features/ui/index.js Outdated Show resolved Hide resolved
@Gargron Gargron force-pushed the fix-remove-conversations-column branch 2 times, most recently from 743c217 to 1da7ed4 Compare April 28, 2022 21:58
- Change name of conversations column in web UI
- Change hotkey for opening conversations column from `g d` to `g c` in web UI
- Remove shortcuts for creating direct-visibility statuses from web UI
@Gargron Gargron force-pushed the fix-remove-conversations-column branch from 1da7ed4 to a2ccd27 Compare April 28, 2022 22:19
@Gargron Gargron merged commit be0bc9a into main Apr 28, 2022
@Gargron Gargron deleted the fix-remove-conversations-column branch April 28, 2022 22:24
@leahoswald
Copy link

leahoswald commented Apr 29, 2022

As and admin (for chaos.social) I want to vote against this change in the current form. The PR is missing a full description for a lot of changes, for example renaming toots to posts in a lot of places. It makes things not more clear it is more confusing and a bad decision from an UI/UX point of view. Also doing such a big change without any discussion is really bad practice.

Don't get me wrong, we could discuss all these changes but not merged in one PR that does look like something else.

@koyuawsmbrtn
Copy link
Contributor

koyuawsmbrtn commented Apr 29, 2022

Renaming toot to post in a lot of places has already been done. The change was actually there to avoid confusion, because "toot" isn't a really a name outside of Mastodon. When I say "I am tooting now" nobody would know that I mean Mastodon. It makes sense.

The change from "Direct" to "Only people I mention" comes from the official Mastodon app as end-to-end encrypted messages are planned. It makes sense, because it implies what it does and doesn't confuse people who expect an actual direct message UI. I could see the "Conversations" tab getting replaced with the chat system as well.

@leahoswald
Copy link

Okay, maybe the "toot changes" were just bad commit practice then.

Also the change of the name could be discusses I'm just concerned abut the other changes @ClearlyClaire mentioned.

@Gargron
Copy link
Member Author

Gargron commented Apr 29, 2022

  • All "toots" have already been renamed to "posts" in previous stable releases, this just cleans up some code
  • I would not classify it as a big change since it's just changing UI labels and removing 2 replaceable dropdown options
  • This is fixing a rather dire usability issue -- people treating Mastodon posts as DMs and having wrong expectations as a result

See #18079

Edit: You might be unaware that Claire's comment referred to a previous state of the PR and that the PR had been updated since.

@leahoswald
Copy link

This is part of the info I hoped for to find in the PR.

@n0toose
Copy link
Contributor

n0toose commented Apr 29, 2022

As I was the person that originally opened the issue, I am just sharing a concern that the way this is now worded makes people excessively paranoid, but on the other hand, it's better than nothing at all and it's probably going to get superseded by some E2EE functionality in less than the 4 years it took for the text to get updated, without allowing any fear get displaced onto random instance owners. 😄 👍🏻

Thanks for addressing this! ⭐

Gargron added a commit that referenced this pull request May 3, 2022
Gargron added a commit that referenced this pull request May 3, 2022
kadoshita pushed a commit to kadoshita/mastodon that referenced this pull request May 4, 2022
mastodon#18146)

- Change name of conversations column in web UI
- Change hotkey for opening conversations column from `g d` to `g c` in web UI
- Remove shortcuts for creating direct-visibility statuses from web UI
kadoshita pushed a commit to kadoshita/mastodon that referenced this pull request May 4, 2022
ClearlyClaire pushed a commit to ClearlyClaire/mastodon that referenced this pull request Jul 6, 2023
mastodon#18146)

- Change name of conversations column in web UI
- Change hotkey for opening conversations column from `g d` to `g c` in web UI
- Remove shortcuts for creating direct-visibility statuses from web UI
ClearlyClaire pushed a commit to ClearlyClaire/mastodon that referenced this pull request Jul 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants