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

Click on the channel's topic to edit it #277

Closed
TheSonAlsoRises opened this issue Apr 25, 2016 · 9 comments · Fixed by #3349
Closed

Click on the channel's topic to edit it #277

TheSonAlsoRises opened this issue Apr 25, 2016 · 9 comments · Fixed by #3349
Labels
help wanted Tickets the community can help us with, by either answering questions or sending us PRs. Type: Feature Tickets that describe a desired feature or PRs that add them to the project.
Milestone

Comments

@TheSonAlsoRises
Copy link

You can view, but not directly edit, a channel's topic from The Lounge. It would be nice if you could click on a topic and get an edit prompt so you can change it.

@AlMcKinlay AlMcKinlay added the Type: Feature Tickets that describe a desired feature or PRs that add them to the project. label Apr 25, 2016
@MaxLeiter
Copy link
Member

Working on this now, should the titlebar be a content-editable, should a modal appear, or should the message box be set to /title <title>? Or something else?

@AlMcKinlay
Copy link
Member

I would say the topic in the title should be content-editable. It's the most user friendly. Modals work for some things (for example, ban list, when we come to that) but it's overkill for this.

When editing, return should save and esc should cancel.

I think onBlur should probably save, but I'm not 100% sure about that. I'm thinking that it would be confusing to me if it didn't save when I clicked out of it. We should probably have a visible way of cancelling the edit, though. Not sure on that one.

@MaxLeiter
Copy link
Member

Not sure if the content-editable is great due to mobile and titles not wrapping. I think adding a context menu option to launch a modal is the way to go.

For reference, this context menu:
this

@maxpoulin64
Copy link
Member

Thing to keep into consideration: topics can contain colors and other codes, so content-editable probably isn't a good idea for that. I would rather just swap the topic with an <input> box on click, and save when it loses focus and the topic has changed. That way it's formatting safe and you get the scrolling and everything for free.

@astorije
Copy link
Member

I agree with @maxpoulin64 except for the save. Saving shouldn't apply without the user consent, aka clicking on a ✅ button.
Example of negative side effect: setting a topic that requires a URL, you start typing, click somewhere else on the chat to select a URL someone gave, boom, too late, topic updated. Not very user-friendly.
Also, rather than the text being clickable, an edit button would be nicer. Opposite can collide with an existing URL, for example if the topic is a URL only.
I'm thinking the edit button should appear within the entries of the dropdown menu.

@AlMcKinlay
Copy link
Member

Would doing the same as what the username has now work well enough?

@PolarizedIons
Copy link
Contributor

A way to just view the entire topic would also be nice. Some channel topics (especially on mobile) are cut off and the only way to view it currently is to hover over it, and a tooltip appears. This isn't nice if you want to click a link though

@MaxLeiter
Copy link
Member

Short term you can just use /topic

@astorije
Copy link
Member

A way to just view the entire topic would also be nice. Some channel topics (especially on mobile) are cut off and the only way to view it currently is to hover over it, and a tooltip appears. This isn't nice if you want to click a link though

This is a different topic, but there is a PR opened about tooltips right now so why not. @xPaw, what about adding this in your timestamp tooltip PR (#824)? It might look super ugly though, so just an idea.

Would doing the same as what the username has now work well enough?

@maxpoulin64 says no, but I think it's worth giving it a shot.

@astorije astorije added the help wanted Tickets the community can help us with, by either answering questions or sending us PRs. label Jan 23, 2018
@xPaw xPaw added this to the 3.2.0 milestone Aug 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Tickets the community can help us with, by either answering questions or sending us PRs. Type: Feature Tickets that describe a desired feature or PRs that add them to the project.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants