Skip to content

Remove channel subtitles from UI#428

Merged
zjs81 merged 1 commit into
zjs81:devfrom
sethoscope:remove-channel-subtitles
May 8, 2026
Merged

Remove channel subtitles from UI#428
zjs81 merged 1 commit into
zjs81:devfrom
sethoscope:remove-channel-subtitles

Conversation

@sethoscope

Copy link
Copy Markdown
Contributor

Per issue #418, this commmit removes channel subtitles from the channel list and from the map screen (deep in the marker sharing). This reduces visual clutter and allows for more compact lists, and the type of channel is already indicated by the leading icon.

The subtitles simply said "Public channel", "Hashtag channel", or "Private channel".

We also remove the relevant localization strings.

@446564 446564 self-assigned this May 6, 2026
@446564

446564 commented May 6, 2026

Copy link
Copy Markdown
Collaborator

I feel like they need a little more padding top and bottom, in the cards. looks squished now

Per issue zjs81#418, this commmit removes channel subtitles from the channel
list and from the map screen (deep in the marker sharing). This reduces
visual clutter and allows for more compact lists, and the type of
channel is already indicated by the leading icon.

The subtitles simply said "Public channel", "Hashtag channel", or
"Private channel".

We also remove the relevant localization strings.
@sethoscope sethoscope force-pushed the remove-channel-subtitles branch from f0b10d7 to ccd23c4 Compare May 6, 2026 21:07
@sethoscope

Copy link
Copy Markdown
Contributor Author

I agree, it was a bit cramped. I added more/different padding.
I rebased & squashed so you still get just one commit.

screenshot

@446564 446564 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks much cleaner

@zjs81 zjs81 merged commit 08ac60a into zjs81:dev May 8, 2026
6 checks passed
@sethoscope sethoscope deleted the remove-channel-subtitles branch May 8, 2026 21:42
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.

3 participants