-
Notifications
You must be signed in to change notification settings - Fork 87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hide user cursors #4126
Comments
I like the idea to always show the cursors but only display the names on hover and while they're typing (with a timeout, maybe something like 3-5 seconds). |
Please add an option to completely disable the user cursors. I've just been in a meeting where a lot of people were annoyed by all the user cursors blinking around (optical noise). |
I agree it should be possible to hide them. I wonder if it would be a middle ground to only show them when people are actually typing - but that also seems harder to accomplish. |
It would be ideal if they were only shown when typing, but as a quick fix we could have a checkbox for it in the top right active users menu called "Show collaborator cursors" (Show author cursors? Cc @jancborchardt for wording?) right below "Show author colors" :) |
I would also say they should only show when typing, as @max-nextcloud proposed. Adding an option is not really a quick fix, because:
|
@jancborchardt Good point about removing the setting :) for 1) it is enabled by default and people can switch it off from there, and for 3) it wouldn't be a per document setting, ideally? it would be a per user setting so once you unset it it would remain like that. What do you think? @max-nextcloud @mejo- would this work? |
I think showing the user cursor labels when typing is NOT a good middle ground. Maybe showing a pure colored cursor without label is OK. If you really want to have some feature like this, do it like Etherpad Lite. Coloring the text background and provide some user list view at the side.
Please consider: And additionally: |
If we don't receive an update from a user within five seconds, hide the label of their cursor. Implementation details: * Add CSS to fade out the cursor label after five seconds per default. * Listen for Yjs updates. In case it's an awareness message, update the cursor of the author, remove the fade out CSS class and add it again after five seconds. Fixes: #4126 Signed-off-by: Jonas <jonas@freesources.org>
If we don't receive an update from a user within five seconds, hide the label of their cursor. Implementation details: * Add CSS to fade out the cursor label after five seconds per default. * Listen for Yjs updates. In case it's an awareness message, update the cursor of the author, remove the fade out CSS class and add it again after five seconds. Fixes: #4126 Signed-off-by: Jonas <jonas@freesources.org>
If we don't receive an update from a user within five seconds, hide the label of their cursor. Implementation details: * Add CSS to fade out the cursor label after five seconds per default. * Listen for Yjs updates. In case it's an awareness message, update the cursor of the author, remove the fade out CSS class and add it again after five seconds. Fixes: #4126 Signed-off-by: Jonas <jonas@freesources.org>
Implementation details: * Add a custom CollaborationCursor Tiptap extension * Add CSS to fade out the cursor label after some time. * Listen for Yjs updates. - If it's a doc change by ourself, update the timestamp of own user in awareness state. - If it's a remote awareness update, add back the CSS class to the corresponding cursor. - Wait 50ms before showing the cursor in the DOM to account for cases where the cursor gets re-rendered by y-prosemirror. Fixes: #4126 Signed-off-by: Jonas <jonas@freesources.org>
Implementation details: * Add a custom CollaborationCursor Tiptap extension * Add CSS to fade out the cursor label after some time. * Listen for Yjs updates. - If it's a doc change by ourself, update the timestamp of own user in awareness state. - If it's a remote awareness update, add back the CSS class to the corresponding cursor. - Wait 50ms before showing the cursor in the DOM to account for cases where the cursor gets re-rendered by y-prosemirror. Fixes: #4126 Signed-off-by: Jonas <jonas@freesources.org>
Implementation details: * Add a custom CollaborationCursor Tiptap extension * Add CSS to fade out the cursor label after some time. * Listen for Yjs updates. - If it's a doc change by ourself, update the timestamp of own user in awareness state. - If it's a remote awareness update, add back the CSS class to the corresponding cursor. - Wait 50ms before showing the cursor in the DOM to account for cases where the cursor gets re-rendered by y-prosemirror. Fixes: #4126 Signed-off-by: Jonas <jonas@freesources.org>
Implementation details: * Add a custom CollaborationCursor Tiptap extension * Add CSS to fade out the cursor label after some time. * Listen for Yjs updates. - If it's a doc change by ourself, update the timestamp of own user in awareness state. - If it's a remote awareness update, add back the CSS class to the corresponding cursor. - Wait 50ms before showing the cursor in the DOM to account for cases where the cursor gets re-rendered by y-prosemirror. Fixes: #4126 Signed-off-by: Jonas <jonas@freesources.org>
Implementation details: * Add a custom CollaborationCursor Tiptap extension * Add CSS to fade out the cursor label after some time. * Listen for Yjs updates. - If it's a doc change by ourself, update the timestamp of own user in awareness state. - If it's a remote awareness update, add back the CSS class to the corresponding cursor. - Wait 50ms before showing the cursor in the DOM to account for cases where the cursor gets re-rendered by y-prosemirror. Fixes: #4126 Signed-off-by: Jonas <jonas@freesources.org>
Implementation details: * Add a custom CollaborationCursor Tiptap extension * Add CSS to fade out the cursor label after some time. * Listen for Yjs updates. - If it's a doc change by ourself, update the timestamp of own user in awareness state. - If it's a remote awareness update, add back the CSS class to the corresponding cursor. - Wait 50ms before showing the cursor in the DOM to account for cases where the cursor gets re-rendered by y-prosemirror. Fixes: #4126 Signed-off-by: Jonas <jonas@freesources.org>
User cursors might be in the way for reading text. Ideally we only show the names on hover or when people are editing. Other than that a checkbox to disable them in the collaborator menu might be worth as well.
What do you think @max-nextcloud @nimishavijay ?
The text was updated successfully, but these errors were encountered: