gui: improve channel selection behavior #21
Reference in New Issue
Block a user
Delete Branch "improve-channel-selection-behavior"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
This change improves the channel selection behavior to be more similar to the official client and generally more usable. It's currently mildly broken due to the details element grabbing click events from the whole row and the row text being selectable. This change also makes it more obvious that the channel title can be clicked. I'm not sure how this works on mobile, so we might need to make more changes there in the future to work better with touchscreens.
Changes
Testing
I tested on the desktop client. I didn't test on mobile but I'll give it a shot after I merge and maybe come back with another PR to make this behavior more intuitive over there.
gui: Improve channel selection behaviorto gui: improve channel selection behavior@@ -86,0 +101,4 @@visibility: hidden;}/* The whole right side of the row is the dblclick target */We might want to make this a single click on mobile. In my experience double clicks (especially if there are a lot of nearby elements) can be awkward on mobile.
I agree that double click is bad ux on mobile. Maybe we can just have a button here that appears only on mobile? Honestly it might prove to be nice to have on desktop as well.
I can confirm that it behaves as I expect. Overall it LGTM, but none of this is my specialty ¯_(ツ)_/¯