forked from wangqr/Aegisub
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Video panning #3
Open
MIRIMIRIM
wants to merge
25
commits into
mod
Choose a base branch
from
video-panning
base: mod
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Commits on Feb 20, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 6285e4e - Browse repository at this point
Copy the full SHA 6285e4eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 4b59099 - Browse repository at this point
Copy the full SHA 4b59099View commit details -
Configuration menu - View commit details
-
Copy full SHA for 51dafe2 - Browse repository at this point
Copy the full SHA 51dafe2View commit details -
Fix mouse position bug when videoAr > displayAr
Crosshair drawing is broken when videoAr != displayAr
Configuration menu - View commit details
-
Copy full SHA for d78dace - Browse repository at this point
Copy the full SHA d78daceView commit details -
I'm not sure about that scale_factor, i don't know how to test that
Configuration menu - View commit details
-
Copy full SHA for bb65a63 - Browse repository at this point
Copy the full SHA bb65a63View commit details -
Configuration menu - View commit details
-
Copy full SHA for 41d8819 - Browse repository at this point
Copy the full SHA 41d8819View commit details -
Configuration menu - View commit details
-
Copy full SHA for da7154b - Browse repository at this point
Copy the full SHA da7154bView commit details -
Configuration menu - View commit details
-
Copy full SHA for b385aa0 - Browse repository at this point
Copy the full SHA b385aa0View commit details -
Configuration menu - View commit details
-
Copy full SHA for b953c92 - Browse repository at this point
Copy the full SHA b953c92View commit details -
with a "good enough" implementation
Configuration menu - View commit details
-
Copy full SHA for 1c3c9e3 - Browse repository at this point
Copy the full SHA 1c3c9e3View commit details -
The preview window would change in width, when the video was being zoomed. Possibly, because on windows, zooming only calls UpdateSize once, but on linux, it gets called multiple, like 8 times. Rounding fixed it.
Configuration menu - View commit details
-
Copy full SHA for 57883bf - Browse repository at this point
Copy the full SHA 57883bfView commit details -
I'm not entirely sure, that this is how it's supposed to be added. This requires editing the already existing hotkey.json file (shows path in log, if it exists). Should this also be added to hotkey.cpp?
Configuration menu - View commit details
-
Copy full SHA for b2dd7fe - Browse repository at this point
Copy the full SHA b2dd7feView commit details -
Revert "Add reset hotkey as Ctrl-R" / Not needed
This reverts commit 3d525d9. The default hotkey is not required. People can just add a custom one from the hotkey settings if they want this.
Configuration menu - View commit details
-
Copy full SHA for e8a19fd - Browse repository at this point
Copy the full SHA e8a19fdView commit details -
Configuration menu - View commit details
-
Copy full SHA for 66e0a1d - Browse repository at this point
Copy the full SHA 66e0a1dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7bdad68 - Browse repository at this point
Copy the full SHA 7bdad68View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3308161 - Browse repository at this point
Copy the full SHA 3308161View commit details -
Configuration menu - View commit details
-
Copy full SHA for 527b838 - Browse repository at this point
Copy the full SHA 527b838View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6287089 - Browse repository at this point
Copy the full SHA 6287089View commit details -
Configuration menu - View commit details
-
Copy full SHA for 935d546 - Browse repository at this point
Copy the full SHA 935d546View commit details -
video zoom: Fix aspect ratio forcing
Do this by partially reverting the changes to UpdateSize in favor of just multiplying by videoZoomValue *after* setting all the window sizes.
Configuration menu - View commit details
-
Copy full SHA for d673f0a - Browse repository at this point
Copy the full SHA d673f0aView commit details -
Configuration menu - View commit details
-
Copy full SHA for c85e5d2 - Browse repository at this point
Copy the full SHA c85e5d2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7eb7eb7 - Browse repository at this point
Copy the full SHA 7eb7eb7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7ab5bf0 - Browse repository at this point
Copy the full SHA 7ab5bf0View commit details -
It's probably stable enough at this point, and hiding it behind an option just creates more confusion than it prevents.
Configuration menu - View commit details
-
Copy full SHA for 1593b9d - Browse repository at this point
Copy the full SHA 1593b9dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0d6fc11 - Browse repository at this point
Copy the full SHA 0d6fc11View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.