Image-sequencer: Add x and y inputs to Resize module

[jywarren]

We have a great new Resize module by @KusioDev!

Try it here: http://sequencer.publiclab.org/examples/#steps=resize{resize:25%25}

It's super. Let's expand on it by allowing resize to be done either with locked aspect ratio (the current, where both x and y are scaled the same) or separately by x and y:

See the resize code here: https://github.com/publiclab/image-sequencer/tree/main/src/modules/Resize/

[jywarren]

This has been marked as a good candidate for becoming a first-timers-only issue like these, meaning that it's simple, self-contained, and with some extra formatting, could be a great entry point for a new contributor. If you're familiar enough with this code, please consider reformatting or reposting it as a first-timers-only issue, and then ping @publiclab/reviewers to get it labelled. Or, if this is not your first time, try to solve it yourself!

---

[gitmate[bot]]

GitMate.io thinks possibly related issues are https://github.com/publiclab/image-sequencer/issues/435 (Resize/scale module), https://github.com/publiclab/image-sequencer/issues/205 (add "drag to crop" UI feature to Crop module), https://github.com/publiclab/image-sequencer/issues/138 (Add a Dithering module), https://github.com/publiclab/image-sequencer/issues/157 (Add a Contrast module), and https://github.com/publiclab/image-sequencer/issues/233 (Add an "import image" module).

[gitmate[bot]]

GitMate.io thinks possibly related issues are https://github.com/publiclab/image-sequencer/issues/435 (Resize/scale module), https://github.com/publiclab/image-sequencer/issues/205 (add "drag to crop" UI feature to Crop module), https://github.com/publiclab/image-sequencer/issues/138 (Add a Dithering module), https://github.com/publiclab/image-sequencer/issues/157 (Add a Contrast module), and https://github.com/publiclab/image-sequencer/issues/233 (Add an "import image" module).

[VibhorCodecianGupta]

@jywarren can I resolve this one?

[jywarren]

haha - one at a time i guess but we'd love your help on all these :-)

On Wed, Dec 5, 2018 at 10:47 AM Vibhor Gupta [e-mail]
wrote:

@jywarren https://github.com/jywarren can I resolve this one?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/image-sequencer/issues/464#issuecomment-444532668,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJ55vRn8zmIJxYauA8rVhQWz7NOaAks5u1-pkgaJpZM4YYllw
.

[VibhorCodecianGupta]

Yes, sure!

[harshkhandeparkar]

Is this resolved yet? Is anyone already working on this?

[VibhorCodecianGupta]

Yup, I'm working on this!

[harshkhandeparkar]

Ok. NP.

[Mridul97]

Hi, @VibhorCodecianGupta are you working on this issue? Can I work on this issue?

[tech4GT]

Hey Mridul I think let's keep it as first timers only! Thanks :)

[VibhorCodecianGupta]

@tech4GT #571

[jywarren]

@Divy123 @aashna27 @HarshKhandeparkar @MargaretAN9 would anyone like to convert this into an FTO?

[aashna27]

Yeah i ll do that by tomorrow.

On Thu, Aug 15, 2019, 10:12 PM Jeffrey Warren [e-mail]
wrote:

@Divy123 https://github.com/Divy123 @aashna27
https://github.com/aashna27 @HarshKhandeparkar
https://github.com/HarshKhandeparkar @MargaretAN9
https://github.com/MargaretAN9 would anyone like to convert this into
an FTO?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/image-sequencer/issues/464?email_source=notifications&email_token=AGKQ7SEPHDUTUR3CP244NPLQEWBQLA5CNFSM4GDCLFYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4MKYMQ#issuecomment-521710642,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGKQ7SBDZYWE33IKV6D2BHDQEWBQLANCNFSM4GDCLFYA
.

[jywarren]

awesome! The existing PR is a great starting point!

On Thu, Aug 15, 2019 at 12:56 PM aashna27 [e-mail] wrote:

Yeah i ll do that by tomorrow.

On Thu, Aug 15, 2019, 10:12 PM Jeffrey Warren [e-mail]
wrote:

@Divy123 https://github.com/Divy123 @aashna27
https://github.com/aashna27 @HarshKhandeparkar
https://github.com/HarshKhandeparkar @MargaretAN9
https://github.com/MargaretAN9 would anyone like to convert this into
an FTO?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/publiclab/image-sequencer/issues/464?email_source=notifications&email_token=AGKQ7SEPHDUTUR3CP244NPLQEWBQLA5CNFSM4GDCLFYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4MKYMQ#issuecomment-521710642
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AGKQ7SBDZYWE33IKV6D2BHDQEWBQLANCNFSM4GDCLFYA

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/image-sequencer/issues/464?email_source=notifications&email_token=AAAF6J4MN4UOBDHCNFTQYQTQEWDAXA5CNFSM4GDCLFYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4ML2BY#issuecomment-521714951,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAF6J4UVZ36HXJRA7YDZ4DQEWDAXANCNFSM4GDCLFYA
.