Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Add avatars below cursors #530

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

KoltesDigital
Copy link

This is not a PR yet, but rather a first step to ask:

  • Is this feature wanted?
  • How do you feel about adding a config to enable/disable this feature?
  • How the avatars could be bound to the editor area?
  • How could it be automatically tested?

Description of the Change

This features adds the site avatar below its cursor. To do so, it uses TextEditor.decorateMarkerLayer with type overlay.

The fact that it's added into the existing markerLayersBySite system has the benefit that the suppression on site disconnection is already handled.

Alternate Designs

Other avatars are displayed in React elements, but it was more complicated to follow the cursor. I believe it perfectly fits as a use case of overlay decoration.

Benefits

The participants follow more easily what others do.

Possible Drawbacks

I saw the avatars are drawn above the bottom bar.

Verification Process

Only manually tested for now.

@sadick254
Copy link

This could make the UI too cluttered. I would suggest you look into teletype and confirm if your suggestion can be a package.
Thanks for the interest. We might review this when the idea is fully fleshed out.

@KoltesDigital
Copy link
Author

You can see what it would look like: https://www.twitch.tv/videos/711181860

Agreed, I haven't though about the teletype provided service. I'll have a look at it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants