Skip to content

Comments

Add the ability to long-press on a channel tab to bring up the 4 channel actions (open/remove/report/block)#1024

Draft
FlaminSarge wants to merge 2 commits intoflex3r:developfrom
FlaminSarge:tab-longpress
Draft

Add the ability to long-press on a channel tab to bring up the 4 channel actions (open/remove/report/block)#1024
FlaminSarge wants to merge 2 commits intoflex3r:developfrom
FlaminSarge:tab-longpress

Conversation

@FlaminSarge
Copy link

@FlaminSarge FlaminSarge commented Dec 29, 2025

This is a proof of concept as I'm mildly certain I should not be setting up the long-click listener here and there is probably a better way to handle the dialog items/etc, but the basic functionality is there.

Please feel free to push further commits to this branch as needed.

Demo (I have since fixed 'Block channel' showing up when logged out):
Screen_recording_20251229_024358.webm

@flex3r flex3r marked this pull request as ready for review February 12, 2026 17:04
@flex3r flex3r marked this pull request as draft February 12, 2026 17:05
@flex3r
Copy link
Owner

flex3r commented Feb 12, 2026

I don't have much time to work on DankChat at the moment but I have a WIP branch where I migrate most of the remaining UI to Compose, so this would be obsolete anyways. But I'm not sure if I like this UX too much, to be honest

@FlaminSarge
Copy link
Author

FlaminSarge commented Feb 13, 2026

I don't have much time to work on DankChat at the moment but I have a WIP branch where I migrate most of the remaining UI to Compose, so this would be obsolete anyways. But I'm not sure if I like this UX too much, to be honest

Re: Compose migration, makes sense.

Regarding the UX, I don't think it's great UX to have to go through 4 (or 5 if that tab isn't currently selected) interactions just to remove a channel tab (tap channel tab, tap 3-dot icon, tap channel, tap Remove Channel, tap Remove on confirmation dialog). I hope the Compose migration streamlines that, because it's currently pretty bad.

I'm going to see about rebasing this against the Compose branch, but please seriously consider reworking the menu UX (if the Compose branch hasn't already done so, of course).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants