Skip to content
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

Improve mobile comment marker in Impress #10964

Open
eszkadev opened this issue Jan 20, 2025 · 2 comments
Open

Improve mobile comment marker in Impress #10964

eszkadev opened this issue Jan 20, 2025 · 2 comments
Labels
enhancement New feature or request impress Issues/Changes that affect impress specifically phone Relevant to mobile phone view of Online unconfirmed UX

Comments

@eszkadev
Copy link
Contributor

Describe the Bug

Currently we show annotation marker on a slide which cannot be hidden and covers the content.
Maybe we should move it to the comments menu ?
cc: @pedropintosilva @banobepascal

Screenshots

Proposal:

Image

@eszkadev eszkadev added enhancement New feature or request impress Issues/Changes that affect impress specifically phone Relevant to mobile phone view of Online UX labels Jan 20, 2025
@pedropintosilva
Copy link
Contributor

pedropintosilva commented Jan 21, 2025

Thanks @eszkadev for reporting it. However, I'm not sure about that since part of the comment workflow is to reposition the marker, example: here I have repositioned it near to the slide title

Image

So, removing the marker doesn't seem a good solution. Nevertheless I have noticed that those assets (marker) do not change size even if the user zoom out our in. It would be nice if those would look smaller if the zoom is less (but we would need to have a minimum size and probably it would be good to keep the active area a bit bigger than the icon itself in that case)

@eszkadev
Copy link
Contributor Author

Ah, I didnt know. So the plan is that the comment marker will be glued to commented object?
I had impression Impress can have only comment "for slide" not for particular element.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request impress Issues/Changes that affect impress specifically phone Relevant to mobile phone view of Online unconfirmed UX
Projects
Status: No status
Development

No branches or pull requests

2 participants