Compact Design for main Image Module

[cypherean]

Fixes #592

Tablet:

Phone:

Laptop:

• tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with grunt jasmine
• code is in uniquely-named feature branch and has no merge conflicts
• PR is descriptively titled
• PR body includes fixes #0000-style reference to original issue #
• ask @publiclab/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

We have a loose schedule of reviewing and pulling in changes every Tuesday and Friday, and publishing changes on Fridays.

Thanks!

[gitpod-io]

[cypherean]

Please review @jywarren @sagarpreet-chadha

[jywarren]

This looks really great, @shreyaa-sharmaa !!! I wonder if we can get the buttons to be side-by-side on mobile, what do you think?

I also wonder, is it possible to have the "remove file" button only appear once there is an image? This maybe could also be done in CSS, i guess, or with $('.button').hide() maybe?

Thanks, this is great!!!

[cypherean]

If we place the buttons side by side then where should we place the progress bar and filename? Above the buttons in both cases?
I'll make the changes to remove file button.

[cypherean]

Hides remove file button when not in use. Also hides the filename when file is removed.
GIF of the changes:

[jywarren]

Awesome, and yes, that sounds great regarding the progress bar! Thanks so much!!!

[cypherean]

Does this look okay @jywarren ?

[cypherean]

Mobile:

Also noting the slight change in position of filename now:

[jywarren]

It looks tremendous. @shreyaa-sharmaa this is really excellent work. Merging now and releasing the next version!

[jywarren]

Hi @shreyaa-sharmaa - now that this is merged, can you be sure that the changes are also implemented in plots2's matching templates? Thank you!!!

[jywarren]

Hi @shreyaa-sharmaa i'd love to publish this on plots2 if you have any time today! I know it's probably late there. If not, we can push through a small new publication later in the week. Thanks so much for your help!

[cypherean]

On it @jywarren!

[cypherean]

Linking it here publiclab/plots2#8327