Skip to content

Fix: Instead of switch "( $max_aspect_ratio )" use a formula for recalculation aspect ratio on Timeline page #4312

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

Merged
merged 2 commits into from
Apr 21, 2025

Conversation

IgorA100
Copy link
Contributor

In general, we need to somehow calculate the values ​​through a formula.
But I still haven't figured out which formula to use....

@connortechnology
Copy link
Member

Maybe there is a better way. This was done to center the video I think... it was a long time ago. Maybe a fresh approach to layout could do away with it.

@IgorA100
Copy link
Contributor Author

Can be done like on Montage page using gridstack.js

@IgorA100
Copy link
Contributor Author

I looked at the Timeline page code.
It needs a lot of changes to integrate correctly with gridstack.js
I don't have time to do it yet :(

…tion (timeline.css.php)

round(100/$max_aspect_ratio, 2)
@IgorA100 IgorA100 changed the title Fix: Add portrait orientation 1080x1920, 1520x2688 aspect ratio on Timeline page Fix: Instead of switch "( $max_aspect_ratio )" use a formula for recalculation aspect ratio on Timeline page Apr 21, 2025
@IgorA100
Copy link
Contributor Author

@connortechnology
The conversion formula was very simple :)
It was just a matter of doing (100/$max_aspect_ratio) and rounding, rather than specifying specific values ​​for each aspect ratio.

@connortechnology connortechnology merged commit beb1e6e into ZoneMinder:master Apr 21, 2025
16 of 19 checks passed
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.

2 participants