Added the download button

[harshithpabbati]

Fixes #391

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

• tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with npm test
• code is in uniquely-named feature branch and has no merge conflicts
• PR is descriptively titled
• ask @publiclab/is-reviewers for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help, and don't be discouraged if your tests fail at first!

If tests do fail, click on the red X to learn why by reading the logs.

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software

Thanks!

[harshkhandeparkar]

Harshith, please refer to this to add the download functionality

https://github.com/publiclab/image-sequencer/blob/main/examples/lib/defaultHtmlStepUi.js#L238-L244

[harshithpabbati]

Why was that needed as we already have a function to download???
$('#download-btn').click(function() {
$('.step-thumbnail:last()').trigger("click");
return false;
});
I am trying to use #download-btn id to it but i can't do it .

[harshkhandeparkar]

This function that you are talking about will only click the last step image. It will not work if we want to remove the click-to-download functionality of the images(which we are planning to do)

…

On Tue 5 Mar, 2019, 5:37 PM Harshith pabbati, ***@***.***> wrote: Why was that needed as we already have a function to download??? $('#download-btn').click(function() { $('.step-thumbnail:last()').trigger("click"); return false; }); I am trying to use #download-btn id to it but i can't do it . — You are receiving this because you are on a team that was mentioned. Reply to this email directly, view it on GitHub <#834 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AhKOnxXQjfEorshbqFL5gr5h4g5IA9a9ks5vTl4XgaJpZM4beazz> .

[aashna27]

Please provide a screenshot.

[aashna27]

Hey can the conflicts be resolved here ? and a screenshot be made available because I have to proceed with this for one of my tasks so if you allow I would add the download button along with my task.

[harshithpabbati]

@aashna27 you can go with it

[aashna27]

Can we close this then.