Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

[ICU-648-2] Fix scroll pop issue when viewing single images except of svg file type #709

Merged
merged 1 commit into from
Feb 5, 2018
Merged

Conversation

saturninoabril
Copy link
Member

Summary

Fix scroll pop issue when viewing single images except of SVG file type

  • computes dimension of images depending on viewing window
  • remove progress % loading indicator and just inject the image source on image onload. It improves image loading.

Note: Currently, SVG don't have height & width file info. Need to address separately (ticket: ICU-698).

Ticket Link

Jira ticket: ICU-648

Checklist

  • Ran make check-style to check for style errors (required for all pull requests)
  • Ran make test to ensure unit and component tests passed

@saturninoabril saturninoabril added the 2: Dev Review Requires review by a core commiter label Feb 4, 2018
Copy link
Contributor

@grundleborg grundleborg left a comment

Choose a reason for hiding this comment

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

I'm a bit out of my depth reviewing this kind of change to be honest, but as far as I understand it, it looks sensible.

@esethna esethna added this to the v4.7.0 milestone Feb 5, 2018
@hmhealey hmhealey removed the 2: Dev Review Requires review by a core commiter label Feb 5, 2018
@hmhealey hmhealey merged commit 2e4cbad into mattermost:master Feb 5, 2018
@saturninoabril saturninoabril deleted the ICU-648-2 branch February 5, 2018 16:05
@lindalumitchell lindalumitchell added the Tests/Not Needed Does not require new release tests label Feb 6, 2018
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation and removed Changelog/Not Needed Does not require a changelog entry labels Feb 7, 2018
@amyblais amyblais added the Changelog/Done Required changelog entry has been written label Feb 17, 2018
hmhealey added a commit that referenced this pull request Aug 28, 2020
* MM-13020 Make ClientError.message enumerable

* Add comment explaining new behaviour
hmhealey added a commit that referenced this pull request Mar 17, 2021
* MM-13020 Make ClientError.message enumerable

* Add comment explaining new behaviour
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Changelog/Done Required changelog entry has been written Docs/Not Needed Does not require documentation Tests/Not Needed Does not require new release tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants