Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: mobile storage image preview #2230

Merged
merged 1 commit into from Oct 16, 2021
Merged

fix: mobile storage image preview #2230

merged 1 commit into from Oct 16, 2021

Conversation

m1ga
Copy link
Contributor

@m1ga m1ga commented Oct 15, 2021

What does this PR do?

This PR fixes the mobile image preview size of images in Storage

The current image preview looks like this:

After this PR it will look like this:

The change is only in line 415 but my linter removed some trailing spaces 馃槈

Test Plan

tested it in local Appwrite build with Firefox and Chrome

  • Go to Storage
  • Upload an image
  • Change Browser size to a mobile layout (or just a small browser window)
  • open the preview of the image

Related PRs and Issues

Relates #2164

Have you read the Contributing Guidelines on issues?

鉁旓笍 yes

@TorstenDittmann TorstenDittmann merged commit dd88e32 into appwrite:master Oct 16, 2021
@TorstenDittmann
Copy link
Contributor

THANK YOU so much! All changes merged 馃コ
Please reach out to me on our Discord server if you would like to claim your Appwrite swag!

@TorstenDittmann TorstenDittmann added the hacktoberfest-accepted Accepted for Hacktoberfest, will be merged later label Oct 16, 2021
@m1ga m1ga deleted the imageFix branch October 16, 2021 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Accepted for Hacktoberfest, will be merged later
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants