Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Add "Copy Link" to room context menu #6374

Merged
merged 11 commits into from
Jul 16, 2021
Merged

Add "Copy Link" to room context menu #6374

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
8bf5e61
Add "Copy" to room context menu.
novocaine Jul 14, 2021
7f70b98
Add English i18n string
novocaine Jul 14, 2021
2fe5ad5
Slightly refine error message
novocaine Jul 14, 2021
e054af7
Run yarn i18n
novocaine Jul 14, 2021
7464900
Change menu label to Copy Link
novocaine Jul 16, 2021
a1c658f
Set Menu icon to link
novocaine Jul 16, 2021
ff80bbc
Move copy link to be before settings
novocaine Jul 16, 2021
329f1c9
Update src/components/views/rooms/RoomTile.tsx
novocaine Jul 16, 2021
685b592
Make error message consistent with menu title
novocaine Jul 16, 2021
9cb3c71
Merge branch 'develop' of github.com:matrix-org/matrix-react-sdk into…
novocaine Jul 16, 2021
767d970
Run i18n
novocaine Jul 16, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/components/structures/MatrixChat.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1202,9 +1202,9 @@ export default class MatrixChat extends React.PureComponent<IProps, IState> {
const roomLink = makeRoomPermalink(roomId);
const success = await copyPlaintext(roomLink);
if (!success) {
Modal.createTrackedDialog("Unable to copy room", "", ErrorDialog, {
title: _t("Unable to copy room"),
description: _t("Unable to copy the room to the clipboard."),
Modal.createTrackedDialog("Unable to copy room link", "", ErrorDialog, {
Copy link
Member

Choose a reason for hiding this comment

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

the first 2 params here are just for Analytics (Matomo) that we don't really get any useful data out of so don't matter very much btw

title: _t("Unable to copy room link"),
description: _t("Unable to copy a link to the room to the clipboard."),
});
}
}
Expand Down