Skip to content

Timeline Fullscreen Without Ext Monitor Blackouts #86

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 12 commits into from
Apr 26, 2024
Merged

Timeline Fullscreen Without Ext Monitor Blackouts #86

merged 12 commits into from
Apr 26, 2024

Conversation

cparish312
Copy link
Contributor

@cparish312 cparish312 commented Apr 25, 2024

Previously the method of having the timeline view go fullscreen caused external monitors to blackout (and just kinda freakout generally)

I tried to also add in seeing the mac menubar when the user scrolls up but instead an option to exit full screen is given.

Screen.Recording.2024-04-25.at.2.20.54.PM-1.mov

@jasonjmcghee
Copy link
Owner

jasonjmcghee commented Apr 25, 2024

Hey Connor! Congrats on your first PR! Excited to review this when I get a chance.

Could you please include a short video of going full screen with the news approach?

Does it not occlude the menu bar of the recording, does it fill the whole display, and does it not play the "full screening" transition / animation? These were the "requirements" I was attempting to operate under.

@cparish312
Copy link
Contributor Author

Hey Jason! On my end it seems to have almost same behavior as the previous fullscreening method minus the external monitor blacking out. One thing I noticed is there is now an animation showing a scrolling to the screen. This is due to the change from .fullScreenAuxillary to .fullScreenPrimary. Not sure if that counts as ""full screening" transition / animation".

Finding my first stabs at UI work to be.... educational

@jasonjmcghee
Copy link
Owner

jasonjmcghee commented Apr 26, 2024

Just tested this - yeah the new method incurs a pretty significant delay opening the timeline due to the full screen transition. Though it's faster on subsequent opens.

Old Method:

Screen.Recording.2024-04-25.at.10.49.39.PM.mov

New Method:

Screen.Recording.2024-04-25.at.10.43.25.PM.mov

@jasonjmcghee
Copy link
Owner

If nothing else, we could allow toggling between these approaches but I'm hoping there's a happy medium that fixes the issues you identified without needing the full screen window delays.

Making an image / video full screen with preview doesn't have this problem... 🤔

@cparish312
Copy link
Contributor Author

After many iterations, I ended up with the first thing I tried (but this time a working version) 😂

Screen.Recording.2024-04-26.at.12.28.47.PM-1.mov

@jasonjmcghee jasonjmcghee merged commit b2f0a52 into jasonjmcghee:main Apr 26, 2024
@jasonjmcghee
Copy link
Owner

🎉 Congrats on joining as a rem contributer! 🎉

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