Skip to content

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

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

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

Conversation

@Just-Insane
Copy link
Collaborator

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

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
Copy link
Member

dozro commented Mar 16, 2026

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

i think i missed some tho :/

@Just-Insane
Copy link
Collaborator 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
Copy link
Member

7w1 commented Mar 17, 2026

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
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
Collaborator 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