Handle zoom in media details view

[misaochan]

App review:

The "Contributions" section is terribly designed: full-width images with title text over them instead of thumbnails. Tapping to open one instead shows a full-height portion of the image that can't be scrolled across nor zoomed out of. Basically, it's a pain to see what you've uploaded and impossible to see a whole image unless you open it in the browser.

I actually thought that we had redesigned media details to fix this - I recall @ujjwalagrawal17 was working on it during the hackathon, and had a branch going that handled zoom and scrolling appropriately. I also recall we were close to merging, but I can't actually find the PR now. @maskaravivek any idea?

[kbhardwaj123]

I think it would be better if aspect ratio of the image be maintained in displaying details rather than cropping in, moreover on cropping in the quality degrades we can implement something similar to google photos or pinterest mobile app

[maskaravivek]

@misaochan Here's the original issue #2881 and PR #2954 that @ujjwalagrawal17 worked on and which was merged to the hackathon branch. I am not sure if handling zoom was part of the redesign.

[kbhardwaj123]

@misaochan I think we can implement semthing like this simple mockup app:

[misaochan]

@kbhardwaj123 That looks good, would you be interested in implementing the fix? :)

think it would be better if aspect ratio of the image be maintained in displaying details rather than cropping in

I agree.

[kbhardwaj123]

@misaochan yes sure I'll start working on this

[misaochan]

Thanks @kbhardwaj123 . Please update us as you go along, and ping me when the PR is ready. :)

[kbhardwaj123]

@misaochan @maskaravivek I want your opinion on one thing, while working on the Media Detail Fragment It came to my mind that the details of a contribution can be shown as Material Design Cards(shown above in my mockup) rather than the grey Edgy tile design we have right now shown below:

[misaochan]

Yes, the grey tile design is very old. I personally would be in support of switching to material design cards, however I'm not sure we should conflate that with this issue. Perhaps you can create a new issue for it? :)

[kbhardwaj123]

@misaochan sure I'll do that right away