Image-sequencer: Custom Resize Option in Resize Module

[aashna27]

Hi, this is a first-timers-only issue,we've worked to make it more legible to folks who either haven't contributed to our code-base before, or for folks who haven't contributed to open source before.

If that's you, we're interested in helping you take the first step and can answer questions and help you out as you do. Note that we're especially interested in contributions from people from groups underrepresented in free and open source software!

If you have contributed before look through our general help wanted issues. Thanks!

🤔 What you will need to know.

Make yourself familiar with the code-base and the resize module.

📋 Step by Step

• [ ] 🙋 Claim this issue: Comment below. If someone else has claimed it, ask if they've opened a pull request already and if they're stuck -- maybe you can help them solve a problem or move it along!

• [ ] 📝 Update

Currently the resize module accepts a parameter to resize the image and resizes both the image height and width with that. We need to give the user and option to select which mode the user wants, "Custom Resize" or "Proportional Resize"(the current one).

"Custom resize" would resize the image as per the width and height parameters entered.

• [ ] 💾 Commit your changes

• [ ] 🔀 Start a Pull Request. There are two ways how you can start a pull request:

  If you are not familiar with the terminal or would like to learn it, here is a great tutorial on how to send a pull request using the terminal.

• [ ] 🏁 Done Ask in comments for a review :)

Please keep us updated

💬⏰ - We encourage contributors to be respectful to the community and provide an update within a week of claiming a first-timers-only issue. We're happy to keep it assigned to you as long as you need if you update us with a request for more time or help, but if we don't see any activity a week after you claim it we may reassign it to give someone else a chance. Thank you in advance!

If this happens to you, don't sweat it! Grab another open issue.

🔗- We encourage contributors to link to the original issue in their pull request so all users can easily see if someone's already started on it.

🤔❓ Questions?

Leave a comment below!

[aashna27]

Yeah yeah suree please claim it. Do assign yourself!

On Sun, Aug 25, 2019, 10:17 PM Priyanka Bansal [e-mail]
wrote:

Hello @aashna27 https://github.com/aashna27 , can i take this?

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

[BrooksPatola]

is this still open?

[harshkhandeparkar]

I guess yes. Go ahead

[BrooksPatola]

Thanks. I'll give it ago @HarshKhandeparkar

[BrooksPatola]

@HarshKhandeparkar If i'm understanding correctly, you wish to keep the current Proportional Resize option (125% start value) but would like to add a Custom Resize option that would take both a width and heigh percentage then apply that to the image?

[harshkhandeparkar]

yes exactly

[BrooksPatola]

@HarshKhandeparkar Would you have a general outline to go about doing this? Just so I am not missing any steps. If not, I will dive into it soon.

[harshkhandeparkar]

Nope. You can start right away. I would generally start by understanding the current code and play with it and then fix the issue.

[BrooksPatola]

@HarshKhandeparkar

I know I should include custom resize functionality in the resize module itself, but I am having difficulty finding where I should include the html tags for the module. When i run the project with 'npm start', it goes to the example page, then I can use Chrome dev tools to see the current elements, I am just unsure of the file to target to include new values. If you have any suggestions, let me know. Otherwise, I'll continue to play around more with it.

[harshkhandeparkar]

To include new options, you have to change the info.json file inside the
module folder

On Sat, 31 Aug, 2019, 11:29 PM Brooks Patola, [e-mail]
wrote:

@HarshKhandeparkar https://github.com/HarshKhandeparkar

I know I should include custom resize functionality in the resize module
itself, but I am having difficulty finding where I should include the html
tags for the module. When i run the project with 'npm start', it goes to
the example page, then I can use Chrome dev tools to see the current
elements, I am just unsure of the file to target to include new values. If
you have any suggestions, let me know. Otherwise, I'll continue to play
around more with it.

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

[Starkyv]

Is this issue still open?

[Starkyv]

How to Claim the issue?

[Starkyv]

@HarshKhandeparkar

I know I should include custom resize functionality in the resize module itself, but I am having difficulty finding where I should include the html tags for the module. When i run the project with 'npm start', it goes to the example page, then I can use Chrome dev tools to see the current elements, I am just unsure of the file to target to include new values. If you have any suggestions, let me know. Otherwise, I'll continue to play around more with it.

@BrooksPatola Do you need any help in solving this issue? I can help you out if so.

[harshkhandeparkar]

If @BrooksPatola doesn't reply in a few days, I guess you can work on it if you want @Starkyv

[BrooksPatola]

@harshithpabbati Still on it. Will update within the next week.

[harshkhandeparkar]

Np. Take your time. Just let us know if you are still on it. BTW, I am HarshKhandeparkar not harshitpabbati :smiley: . It is a very common typo

[Starkyv]

@harshithpabbati Still on it. Will update within the next week.

Sure @BrooksPatola . Take your time.

[Starkyv]

@harshithpabbati Still on it. Will update within the next week.

@BrooksPatola Can I help you with this issue? If so please enlighten me with the problem you are facing.

[harshkhandeparkar]

If no one is working on this, let's mark this for gci

[keshav234156]

moved to #1298