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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Screenshots are being stretched #470

Closed
btkostner opened this issue Sep 7, 2017 · 3 comments · Fixed by #2121
Closed

Screenshots are being stretched #470

btkostner opened this issue Sep 7, 2017 · 3 comments · Fixed by #2121
Assignees
Labels
Status: Confirmed Verified by someone other than the reporter

Comments

@btkostner
Copy link
Contributor

btkostner commented Sep 7, 2017

Currently:

screenshot from 2017-09-06 19 19 04

Expected (from GitHub README):

screenshot from 2017-09-06 19 19 13

Difference:

appcenter

Confirmed that the screenshot being used is the same and that AppCenter is serving the correct image.
This URL could change, but here is the raw AppStream version:
https://appstream.elementary.io/appcenter/media/pool/com/github/stsdc.monitor.desktop/AC70FDD191D197674F7D0001F16A2BF2/screenshots/image-1_orig.png

@danirabbit danirabbit added the Status: Confirmed Verified by someone other than the reporter label Dec 10, 2018
@davidmhewitt
Copy link
Member

The screenshots in AppCenter are being scaled to fit the maximum width of 800px but not disproportionately as far as I can tell. Aspect ratio seems to be preserved with all the examples I've looked at.

Can someone confirm if this was/is still an issue?

@cassidyjames
Copy link
Contributor

I can confirm from these screenshots that it's not stretched, just scaled while preserving aspect ratio. Is the issue that it's scaling up?

@cassidyjames cassidyjames added Status: Incomplete Needs more information before we can take action and removed Status: Confirmed Verified by someone other than the reporter labels Apr 9, 2020
@danirabbit
Copy link
Member

Yes, instead of scaling up, we should center

@danirabbit danirabbit added Status: Confirmed Verified by someone other than the reporter and removed Status: Incomplete Needs more information before we can take action labels Apr 17, 2020
@danirabbit danirabbit self-assigned this Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Confirmed Verified by someone other than the reporter
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants