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

Two Zoom Changes (Resolution and Extrema) #597

Merged
merged 1 commit into from
Feb 15, 2019

Conversation

baconpaul
Copy link
Collaborator

@baconpaul baconpaul commented Feb 14, 2019

Activate the 125x assets by making them callable in CScalableBitmap,
activating them in win script, and re-rendering the win resource files.

Also add a function "Zoom to Largest" which takes zoom to exactly the
point below where we would warn that you can't zoom any larger for your
screen.

Closes #540: Activate 125% assets
Closes #590: Zoom to Biggest Menu

@baconpaul
Copy link
Collaborator Author

This is all pretty mechanical. @jsakkine or @kzantow if you want to peek appreciate it. I'll probably sweep tomorrow evening eastern unless I hear otherwise. Thanks!

@esaruoho esaruoho changed the title Two Zoom Changes (Resoultion and Largest) Two Zoom Changes (Resolution and Largest) Feb 14, 2019
@esaruoho
Copy link
Collaborator

esaruoho commented Feb 14, 2019

tested with vst2/renoise and au/logic. 125% looks great. renoise handles max res nice.

but logic OTOH:
image

i.e., i can't access Menu now because Surge is way larger than it should be

@baconpaul
Copy link
Collaborator Author

Ahh that's because the window decoration trim is too aggressive. I need to turn it back. Sigh.

@baconpaul
Copy link
Collaborator Author

Pushed a change to max it out at 90% on mac as well as win.

Activate the 125x assets by making them callable in CScalableBitmap,
activating them in win script, and re-rendering the win resource files.

Also add a function "Zoom to Largest" which takes zoom to exactly the
point below where we would warn that you can't zoom any larger for your
screen, and "Zoom to Smallest", so that @sense-amr can do crazy things
with bitwig and Orca.

Moreover on smaller monitors, 95% zoom in logic pro is too large
and eats the menu button due to decoration, so in this change
we move to 90% of screen as max for all oses.

Closes 540: Activate 125% assets
Closes 590: Zoom to Biggest Menu
@baconpaul baconpaul changed the title Two Zoom Changes (Resolution and Largest) Two Zoom Changes (Resolution and Extrema) Feb 14, 2019
@baconpaul baconpaul merged commit 571a3d3 into surge-synthesizer:master Feb 15, 2019
@baconpaul baconpaul deleted the use-125asset-540 branch February 15, 2019 14:43
baconpaul added a commit to baconpaul/surge that referenced this pull request Jul 10, 2019
Activate the 125x assets by making them callable in CScalableBitmap,
activating them in win script, and re-rendering the win resource files.

Also add a function "Zoom to Largest" which takes zoom to exactly the
point below where we would warn that you can't zoom any larger for your
screen, and "Zoom to Smallest", so that @sense-amr can do crazy things
with bitwig and Orca.

Moreover on smaller monitors, 95% zoom in logic pro is too large
and eats the menu button due to decoration, so in this change
we move to 90% of screen as max for all oses.

Closes surge-synthesizer#540: Activate 125% assets
Closes surge-synthesizer#590: Zoom to Biggest Menu

Former-commit-id: ffda50b51e609979f03ac4c306d66aad336f91a2 [formerly 571a3d3]
Former-commit-id: 06ca64ef12ff34c4ac724585d33cb916e06ae7a3
Former-commit-id: 3f1c1540755c77231baf63d749da883cfaa51d56
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