Skip to content

feat: add multiplayer profile images next to fields#1265

Open
jeremydw wants to merge 3 commits into
mainfrom
feat/multiplayer-profile-images
Open

feat: add multiplayer profile images next to fields#1265
jeremydw wants to merge 3 commits into
mainfrom
feat/multiplayer-profile-images

Conversation

@jeremydw

Copy link
Copy Markdown
Member

This PR slightly improves the "multiplayer" UI by adding profile images adjacent to the field each person is using.

  • It also slightly tweaks the "focused" field UI, to reduce the horizontal padding and linkify the field name instead of the invisible anchor.

jeremydw added 2 commits June 20, 2026 11:11
- for multiplayer mode, show the profile image of all users who have a field focused next to the field itself
- also tune the focused/highlighted behavior to remove some of the padding
- improve the highlighted styles of drawers
- linkify the field title instead of the hidden "#" to make it easier to access
@jeremydw

Copy link
Copy Markdown
Member Author

/preview

@github-actions

Copy link
Copy Markdown
Contributor

rootjs.dev Preview

Preview URL: https://rootjs-dev--pr-1265-awri5kbw.web.app
Commit: bd9b249

This preview is built from the PR head and expires in 2 days.

@stevenle

stevenle commented Jun 24, 2026

Copy link
Copy Markdown
Member

thanks, this is pretty cool. a couple of notes:

  • the inactive state is a little off when using opacity, maybe switch to brightness or something else (or re-order z-index so the active ones are on top):
Screenshot 2026-06-24 at 4 23 20 PM
  • in figma, you can click on an avatar to follow a user, i wonder if we should add a click to deeplink to the field the user is looking at (if they are looking at a field)

  • i wonder if we should also color code the users and highlight the field borders with the color matching the user (this might be a little much / too busy, so consider this feedback optional)

- polish deeplink behavior
- desaturate inactive icons
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