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

Incorrect selection processing #1382

Closed
Rusachok opened this issue Jul 19, 2024 · 6 comments
Closed

Incorrect selection processing #1382

Rusachok opened this issue Jul 19, 2024 · 6 comments
Labels
Milestone

Comments

@Rusachok
Copy link

Describe the bug

I have a tour with a lot of panoramas, I often had to move the gallery to the right and left. Sometimes it turned out that a part of the screen was highlighted and the viewer broke after that.
I was able to clearly fix how to repeat it.
Select the title and, with the mouse key pressed, continue to pull up on the viewer. I.e., I hold down the key, select and press only in the middle of the viewer.
After that, dragging the panorama does not work. When clicked, it starts dragging the panorama without holding down the mouse button. And when pressed, it does nothing.
You need to ignore the possibility of highlighting on the viewer.

I hope you can repeat it. The video does not reflect exactly how the viewer behaves when it breaks down.

ScreenRecorderProject22.mp4

Online demo URL

https://vi-tour.ru/s/eyclgqhj/

Photo Sphere Viewer version

5.8.2

Plugins loaded

No response

OS & browser

Win10, Chrome

Additional context

No response

@Rusachok Rusachok added the bug label Jul 19, 2024
@Rusachok
Copy link
Author

We had a similar problem. It is necessary to prohibit the selection of text in the container:

user-select: none;

@mistic100 mistic100 added this to the 5.8.4 milestone Jul 23, 2024
@mistic100
Copy link
Owner

mistic100 commented Jul 23, 2024

This was actually a problem with the markers container, not the navbar nor the gallery

@Rusachok
Copy link
Author

Are you sure? There are no markers on this tour.

@mistic100
Copy link
Owner

That's irrelevant, the markers plugin is loaded and the container is here.

@Rusachok
Copy link
Author

Great!

Copy link

This feature/bug fix has been released in version 5.9.0.

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

No branches or pull requests

2 participants