Skip to content

[FIX] Fix broken uploaded images in cordova app#11573

Closed
ramrami wants to merge 1 commit intoRocketChat:developfrom
ramrami:fix-uploaded-images-cordova
Closed

[FIX] Fix broken uploaded images in cordova app#11573
ramrami wants to merge 1 commit intoRocketChat:developfrom
ramrami:fix-uploaded-images-cordova

Conversation

@ramrami
Copy link
Copy Markdown
Contributor

@ramrami ramrami commented Jul 26, 2018

Uploaded images are not working in cordova build. I just added the missing call to fixCordova function on the image_url.

thks

@ggazzo ggazzo added this to the 0.69.0 milestone Aug 13, 2018
@ggazzo ggazzo requested review from rssilva and tassoevan August 13, 2018 14:46
@ggazzo ggazzo added the area: ui/ux Related to UI/UX, frontend code, accessibility, and user interaction label Aug 13, 2018
Copy link
Copy Markdown
Member

@tassoevan tassoevan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, but I have some observations.

Internally, lazyloadImage template already uses fixCordova helper, but the rendered data-src attribute of the <img> seems to be ignored. I think we need to fix it there, not where lazyloadImage is used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui/ux Related to UI/UX, frontend code, accessibility, and user interaction

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants