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

[BUG] Multi-View: General zoom, rotation and reset rotation buttons are not working #1489

Open
markusweigelt opened this issue Feb 13, 2025 · 2 comments
Assignees
Labels
🐛 bug A non-security related bug.

Comments

@markusweigelt
Copy link
Contributor

markusweigelt commented Feb 13, 2025

Description

Clicking these buttons doesn't trigger any action. There's no JavaScript error either. It seems the function is not implemented for multi view.

Reproduction

Steps to reproduce the behaviour:

  1. Navigate to:
    https://example.com/viewer?tx_dlf%5Baction%5D=main&tx_dlf%5Bcontroller%5D=PageView&tx_dlf%5BdocPage%5D%5B1%5D=5&tx_dlf%5BdocPage%5D%5B2%5D=9&tx_dlf%5BdocPage%5D%5B3%5D=13&tx_dlf%5BdocPage%5D%5B4%5D=15&tx_dlf%5Bdouble%5D=0&tx_dlf%5Bid%5D=https%3A%2F%2Ftest.dfg-viewer.de%2Ffileadmin%2Fcomposition%2FMus.2389-N-9.xml&tx_dlf%5Bmultiview%5D=1&tx_dlf%5Bpage%5D=1&cHash=519b78c80be30d359759090f71db9599
  2. Klick one of the zoom buttons + or -

Expected Behavior

Hide general zoom, rotation and reset rotation buttons in multiview.

Screenshots and Examples

Image

Environment

  • TYPO3 version: [e.g. 11.5.41]
@markusweigelt markusweigelt added the 🐛 bug A non-security related bug. label Feb 13, 2025
@sebastian-meyer
Copy link
Member

I suggest removing those buttons (as well as those for rotating) in multi-view, because it doesn't make much sense to zoom/rotate all views at once. Zooming/rotating a single view is still possible by mouse-wheel and shift+mouse.

@markusweigelt markusweigelt changed the title [BUG] Multi-View: General zoom buttons are not working [BUG] Multi-View: General zoom, rotation and reset rotation buttons are not working Feb 13, 2025
@markusweigelt
Copy link
Contributor Author

Updated the title and description and added the reset rotation button as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug A non-security related bug.
Projects
None yet
Development

No branches or pull requests

3 participants