Skip to content

fix: Quick issues batch 1 — UI/UX polish - #310

Merged
Just-Insane merged 8 commits into
devfrom
fix/quick-issues-batch-1
Mar 17, 2026
Merged

fix: Quick issues batch 1 — UI/UX polish#310
Just-Insane merged 8 commits into
devfrom
fix/quick-issues-batch-1

Conversation

@Just-Insane

@Just-Insane Just-Insane commented Mar 16, 2026

Copy link
Copy Markdown
Contributor

A collection of small UI/UX and interaction fixes across the app.

Changes

Issue Fix
#2 Reduced-motion: add animation-iteration-count: 1 so loading spinners stop after one cycle instead of running indefinitely at a near-zero speed
#44 Account switcher: show a confirmation dialog before signing out of an account
#45 Server picker: prevent iOS from restoring the old server name while the user is actively editing the input field
#52 Message editor: add autoCapitalize="sentences" to respect the OS/keyboard capitalisation setting on mobile
#66 Add account flow: show a "Cancel" button next to "Adding account" so users can abort and return to the main app
#79 / #35 Autocomplete: pressing Enter now selects the highlighted autocomplete item instead of sending the message; first item is highlighted on open, ArrowUp/Down navigate while keeping typing focus in the editor
#69 Autocomplete: focus correctly returns to the message editor after completing a mention or emoji
#103 Theme color: use correct light (#ffffff) / dark (#1b1a21) colours via media-query <meta> tags and update the PWA manifest
#120 Video & audio messages: volume preference is persisted across page loads via localStorage and shared between all media players
#168 Notifications: add "Favicon Dot: Mentions Only" setting — when enabled, the favicon only changes for mentions/keywords, not all unreads

Note

The matrix.to custom base URL change (#67) has been moved to a separate branch: fix/matrix-to-custom-base. It needs more work (hardcoded regex, incoming link parsing).

Files changed

  • src/index.css — reduced-motion animation-iteration-count
  • src/app/pages/auth/ServerPicker.tsx — iOS focus guard for server input
  • src/app/components/editor/Editor.tsx — autoCapitalize on Editable
  • index.html / public/manifest.json — light/dark theme-color meta
  • src/app/pages/client/sidebar/AccountSwitcherTab.tsx — sign-out confirmation dialog
  • src/app/pages/auth/AuthLayout.tsx — cancel button in add-account flow
  • src/app/state/settings.ts + src/app/pages/client/ClientNonUIFeatures.tsx + src/app/features/settings/notifications/SystemNotification.tsx — faviconForMentionsOnly setting
  • src/app/components/media/Video.tsx + src/app/components/message/content/AudioContent.tsx + src/app/components/RenderMessageContent.tsx — shared volume persistence
  • src/app/components/editor/autocomplete/BaseAutocompleteMenu.tsx — data-autocomplete-menu attribute
  • src/app/features/room/RoomInput.tsx — virtual autocomplete selection, arrow key navigation
  • src/app/components/editor/autocomplete/AutocompleteMenu.tsx — selectedIndex state, navigate event listener
  • src/app/components/editor/autocomplete/{Emoticon,UserMention,RoomMention}Autocomplete.tsx — ReactEditor.focus after completion

@dozro

dozro commented Mar 16, 2026

Copy link
Copy Markdown
Member

linked the issues you mentioned in your table, bc silly github

i think i missed some tho :/

@Just-Insane

Copy link
Copy Markdown
Contributor Author

linked the issues you mentioned in your table, bc silly github

Danke!

@Just-Insane
Just-Insane marked this pull request as ready for review March 17, 2026 00:27
@Just-Insane
Just-Insane requested review from 7w1 and hazre as code owners March 17, 2026 00:27
@Just-Insane
Just-Insane enabled auto-merge March 17, 2026 00:31
@7w1

7w1 commented Mar 17, 2026

Copy link
Copy Markdown
Member

Should the changesets maybe be like... 10 separate files instead of one thing?

Also, that matrix.to config won't work. There's a lot more than just that one line... it's hardcoded in a few places, the regex later in the file, and we'd still want outgoing links to use the normal matrix.to, we just need to parse messages we receive to get replaced with the custom matrix.to I think.

@Just-Insane
Just-Insane disabled auto-merge March 17, 2026 00:44
@github-actions

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

Status Preview URL Commit Alias Updated (UTC)
✅ Deployment successful! https://pr-310-sable.raspy-dream-bb1d.workers.dev 097e12d pr-310 Tue, 17 Mar 2026 01:05:07 GMT

@Just-Insane

Copy link
Copy Markdown
Contributor Author

Moved the matrix.to changes into another branch, as that'll be more work. Fixed the changesets, and setup the voice messages to use the same audio as calling.

@Just-Insane
Just-Insane enabled auto-merge March 17, 2026 01:07
@Just-Insane
Just-Insane added this pull request to the merge queue Mar 17, 2026
Merged via the queue into dev with commit 62925c8 Mar 17, 2026
8 checks passed
@Just-Insane
Just-Insane deleted the fix/quick-issues-batch-1 branch March 17, 2026 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants