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

Ability to copy permalinks for rooms and msgs #276

Closed
matrixbot opened this issue Apr 28, 2016 · 4 comments
Closed

Ability to copy permalinks for rooms and msgs #276

matrixbot opened this issue Apr 28, 2016 · 4 comments
Assignees
Milestone

Comments

@matrixbot
Copy link

Created by @ matthew:matrix.org.

@manuroe
Copy link
Member

manuroe commented Apr 29, 2016

Permalinks for msgs has been done (#225).
Need to find some place in the UI for permalinks for rooms.

@manuroe
Copy link
Member

manuroe commented Aug 2, 2016

Need to find some place in the UI for permalinks for rooms.

It seems to be available from Room settings -> Long press on a room address -> Copy Room URL, no?
The code:
https://github.com/vector-im/vector-ios/blob/bbfba54c39252d0fcf46e62b1ba21cf7c1cc4fe6/Vector/ViewController/RoomSettingsViewController.m#L535

@ara4n
Copy link
Member

ara4n commented Aug 2, 2016

Ah! I was totally failing to discover the long press. Is there any
reason we can't have the menu fire on a normal press?

On 02/08/2016 16:56, manuroe wrote:

Need to find some place in the UI for permalinks for rooms.

It seems to be available from Room settings -> Long press on a room
address -> Copy Room URL, no?
The code:
https://github.com/vector-im/vector-ios/blob/bbfba54c39252d0fcf46e62b1ba21cf7c1cc4fe6/Vector/ViewController/RoomSettingsViewController.m#L535


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#276 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABO_vRn8gXbC7ABGmsaDSJE7qUceg1gcks5qb2iagaJpZM4ISPiI.

@giomfo
Copy link
Member

giomfo commented Aug 22, 2016

Presently normal press is used to set the main address.
Suggestion: display on normal press and add "Set as main address" as an option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants