I started work on this one, quite excited about it:
As itâs using select-kit, it will also be very easy for plugins to add options in this menu.
I started work on this one, quite excited about it:
As itâs using select-kit, it will also be very easy for plugins to add options in this menu.
Getting almost there, I have to tweak a little bit some actions but itâs working:
(link to a not flickering source: Cheesecake are awesome until they fall on the ground - Site Feedback - Discourse.webm - Google Drive)
This is now deployed as of:
https://github.com/discourse/discourse/commit/9923829402997c7af3e25121d6d3a345536f3884
Please let me know what you think.
@erlend_sh I didnt implement the âreply to selected postâ feature as itâs not clear to me how it works/and what it brings. Could you elaborate please ?
Going to put this out as public feedback as well cause we have a lot of whispers here that are discussing it and I wanted to have something public.
The menu here needs to change from being âmixed toggle / mixed actionâ to 100% toggles.
It is mega surprising to have this post directly from that menu, my expectation is that a âblueâ button will cause a post and having a context menu sub item post stuff is mega surprising.
All the thing there should do is toggle modes.
âReply as new topicâ toggles you current response into ânew topicâ mode.
And so onâŠ
It also needs to remember previous context so you have âeasy undoâ
For example:
Reply to post #10
Click: reply as new topic
Click: reply to topic
Click: reply to post by @sam
We are back to replying to post #10
If the âblueâ reply button had a downward triangle on it, I would expect it to post right away after you click the little triangle and select something, but I donât feel we want that kind of UI.
Was talking of this with @erlend I get your point and will do it this way. Just explaining my initial position: this was a power feature mode so I wanted everything to be the fastest and not perfectly intuitive. Doing this it will be a mix of intuitive and fast which is fine to me.
Agree with all the feedback so far â one way you could make it faster (on keyboard at least) would be to have some shortcut to open the menu, or tab index after the compose text area. Then if you want fast:
Compose reply,
shortcut to open and focus reply type
downenter (select reply type)
ctrlenter (post)
One big pain though with keyboard shortcuts in the composer is that they require a lot of keyboard mushing eg ctrl
alt
f
, it just ends up being so much work that I usually just reach for the mouse
If you choose something easy odds are super high it clashes.
Fair point. Thatâs why the alternative suggestion was tab indexing. Similar workflow as before but:
Compose,
Tab to focus reply type
Arrows + enter to select reply type
Ctrl enter to post.
Not sure if that would complicate screen readers, but itâs a way around the hand cramping shortcuts.
One small thing: would anyone be opposed to making this stand out slightly more as a button?
This (closer to our default button style)
instead of
love this feature, btw
@david concerning a bug mentioned by @awesomerobot in whispers:
I linked it to the presence plugin which is I think not expecting we could create a new topic when we are browsing a topic. And will then subscribe to a null channel to messageBus in this case here : https://github.com/discourse/discourse/blob/master/plugins/discourse-presence/assets/javascripts/discourse/components/composer-presence-display.js.es6#L79
I think we shouldnât try to subscribe to anything if action is not reply or edit. What do you think ?
That sounds reasonable, thereâs no need for any presence features on new topics.
The code has changed a lot since I wrote it, so I couldnât say offhand where that change would need to be made. Longer term it would definitely be good to move a lot of this presence logic into message-bus as @sam mentioned a while ago
I think it should be quite subtle, so I prefer the current design. This is an extreme power user feature, not something we want to call a lot of extra attention to.
I just pushed a commit which should largely improve the situation. Let me know what you think @sam
Looks so much better!
âToggle whisperâ is missing in the dropdown when I click the topicâs reply button. Itâs only visible after clicking a postâs reply button. Is that intentional?
Should be fixed thanks!
Love the improvements!
BTW, hereâs a PR for the suggestions on tab indexing (here implemented as index 8, after the âcancelâ focus) and adding :focus styling â Something like this would suit my needs as a keyboard user.
That was my first reaction.
I disagree. Replying privately to a public message seems like a fairly useful thing to do (but maybe we donât want to encourage private conversation?), and the only way that I know to do that is with the .
And when I was teaching with Discourse, reply-as-linked-topic was very useful for people to start a new topic with their solution to the exercise.
Correct, we donât want to do that.