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

Feat/6724 mobile handling #1

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

Conversation

Simounet
Copy link
Member

@Simounet Simounet commented Nov 5, 2024

Should fix wallabag/wallabag#2749
I'm not an heavy user of annotations. Would be great having people testing it. There still a little bit of work to do on the UI side. I've just rotated the annotation icon for testing purpose.

ping @nicosomb

@j0k3r
Copy link
Member

j0k3r commented Nov 6, 2024

I've no idea if this is working or not.
Can you share a video or something as before/after?

@Kdecherf
Copy link
Member

Kdecherf commented Nov 6, 2024

I can give it a try but not before next week

@nicosomb
Copy link
Member

@Simounet could you propose us a demo platform? 🙂

@Simounet Simounet force-pushed the feat/6724-mobile-handling branch from 3cd1873 to c700ca4 Compare November 12, 2024 17:45
@Simounet
Copy link
Member Author

I didn't upload the right fix. Should be better now.

@nicosomb Info sent on Matrix

@nicosomb
Copy link
Member

@Simounet I don't see your messages.

@justinryanc
Copy link

Would love to test this but how? Annotations only work on iPad when using the mouse. Touch doesn’t trigger it. This from what I understand fixes it. If you can guide me on how to test this I would be happy to do so. I am using docker image 2.6.10 currently.

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.

Annotations on smartphones & tablets, responsive
5 participants