cloudinary_npm icon indicating copy to clipboard operation
cloudinary_npm copied to clipboard

Upload Widget - Layout broken for 768px wide

Open ShoryuKyzan opened this issue 2 years ago • 3 comments

Bug report for Cloudinary NPM SDK

Before proceeding, please update to latest version and test if the issue persists

Describe the bug in a sentence or two.

Layout broken between 768 and 767 px. Broken for many tablet users most likely.

Issue Type (Can be multiple)

[ ] Build - Can’t install or import the SDK [ ] Babel - Babel errors or cross browser issues [ ] Performance - Performance issues [X] Behaviour - Functions aren’t working as expected (Such as generate URL) [ ] Documentation - Inconsistency between the docs and behaviour [ ] Incorrect Types - For typescript users who are having problems with our d.ts files [ ] Other (Specify)

Steps to reproduce

In Chrome devtools, use device simulator in responsive mode, and choose the bar for 768px.

Error screenshots

image

Browsers

[X] Chrome [ ] Firefox [ ] Safari [X] Other (Edge) [ ] All Didn't test others

Versions and Libraries

Cloudinary_NPM SDK version 1.26.0, 1.30.0 Node - 16.15.0 NPM - 8.5.5

Config Files

n/a, can't share, have confirmed version via package-lock.json tho.

Repository

n/a, proprietary, can't share

ShoryuKyzan avatar Jun 22 '22 05:06 ShoryuKyzan

Hi Joseph,

Thanks for pointing out this issue, we were able to reproduce it.

The issue seems to be with the upload_widget, not the cloudinary_npm SDK. We have opened this as a bug for the upload_widget that is currently being prioritized for a fix.

Hope this helps, please let me know if you have any further questions.

Best regards, John

cloudy408 avatar Jun 22 '22 14:06 cloudy408

I'd like to subscribe to updates on that issue so I know when it's fixed. Tag me if its public? I cant seem to find that repo here.

ShoryuKyzan avatar Jun 22 '22 17:06 ShoryuKyzan

Hi @ShoryuKyzan - Thanks for following up about it. This GitHub thread is linked in our internal ticket and when there are updates to our internal ticket, we will be updating this thread as well so that it's all updated/in-sync.

aleksandar-cloudinary avatar Jun 23 '22 06:06 aleksandar-cloudinary

Hi @ShoryuKyzan, just wanted to notify you that the issue with broken layout should be already fixed. Can you verify that on your side so I can close this ticket (i.e. our communication channel for us)?

cloudinary-pkoniu avatar Dec 13 '22 07:12 cloudinary-pkoniu

@cloudinary-pkoniu I'll definitely put it on my todo list! We worked around it for the time being by just making the container a little wider or narrower when it was at that resolution.

ShoryuKyzan avatar Dec 13 '22 23:12 ShoryuKyzan