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

A first attempt at improving the adjust size approach #66

Closed
wants to merge 2 commits into from

Conversation

baconpaul
Copy link
Collaborator

This is a first shot at making adjust size / set size work.
It naively just applies aspect ratio to with which makes corner
and bottom drags work but not side drags.

An algorithm to work for all drags without knowing the direction
of the user gesture which is stable continues to elude me, but I wanted
at least something more reasonable in teh codebase before I go on vacation.

Addresses #56

This is a first shot at making adjust size / set size work.
It naively just applies aspect ratio to with which makes corner
and bottom drags work but not side drags.

An algorithm to work for all drags without knowing the direction
of the user gesture which is stable continues to elude me, but I wanted
at least something more reasonable in teh codebase before I go on vacation.

Addresses free-audio#56
@baconpaul
Copy link
Collaborator Author

So @jatinchowdhury18 two questions

  1. My first commit here had an error but I force pushed the fix. How do I make actions run with the fix? I pretty clearly include algorithm now in clap juce wrapper and it builds locally but i'm still getting errors?
  2. thoughts on this change? It is at least a placeholder improvement over the goop we have now

@tobiashienzsch
Copy link
Contributor

To rerun a workflow you need to make a second commit on your branch and push normally. GitHub will trigger a new pipeline.

@baconpaul
Copy link
Collaborator Author

To rerun a workflow you need to make a second commit on your branch and push normally. GitHub will trigger a new pipeline.

ahh thanks
azure re triggers on force pushes also so i just instinctively did that

@baconpaul baconpaul closed this Jun 22, 2022
@jatinchowdhury18
Copy link
Collaborator

It's also possible to manually trigger the CI workflow from here:
Screen Shot 2022-06-23 at 9 25 43 AM

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.

3 participants