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

Add fresh screenshots #144

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Doomsdayrs
Copy link

These fresh screenshots provide a better view of VLC, with GNOME4

Newly added, are windowed/full screen variants

@flathubbot
Copy link
Contributor

Started test build 93280

@flathubbot
Copy link
Contributor

Build 93280 failed

@MatMaul
Copy link
Collaborator

MatMaul commented May 30, 2022

I am not against the change, but PNG with a size between 1 and 4MB is a no-go, not everyone is on unrestricted landline fiber :)
Please resize them down a bit and switch to JPG format so we get to a similar size to the current ones.

@Doomsdayrs
Copy link
Author

I am not against the change, but PNG with a size between 1 and 4MB is a no-go, not everyone is on unrestricted landline fiber :) Please resize them down a bit and switch to JPG format so we get to a similar size to the current ones.

Sure thing, Let me get to that

I left a src tar with the original pngs,
 Just in case the source images are needed for maniupulation in the future.
@flathubbot
Copy link
Contributor

Started test build 93981

@Doomsdayrs
Copy link
Author

@MatMaul

I compressed the images the best I could, but the images of the landscape are more complex, thus the compression could only get them down to about 200kb for those two.

In total, the new size of all 4 images is 614 kB

@flathubbot
Copy link
Contributor

Build 93981 failed

@flathubbot
Copy link
Contributor

Started test build 93985

@flathubbot
Copy link
Contributor

Build 93985 failed

@flathubbot
Copy link
Contributor

Started test build 111601

@flathubbot
Copy link
Contributor

Build 111601 failed

@Doomsdayrs
Copy link
Author

@MatMaul ?

@nurupo
Copy link
Contributor

nurupo commented Apr 2, 2024

Is there any point in providing such high-res images at all? From my experience, Flathub shrinks the images to be way too small, e.g. the text in screenshots for my VLC plugin can't even be read on Flathub, which is very annoying.

@MatMaul
Copy link
Collaborator

MatMaul commented Apr 2, 2024

@Doomsdayrs yes sorry, I was cleaning up related appdata stuff and thought about merging your PR on the way.

However @nurupo has a point, while the storage size is fine the DPI is too high and we would not see much of VLC UI once it is scaled to the presentation size used by flathub.org or GNOME Software.

Do you mind taking screenshots at 720p instead? I can't downscale from the PNG since it will not get the UI any bigger :)

I also wanted to ask you if you used free-to-use video content but I googled the title of the videos and everything seems fine on this side.

Sorry for taking so long, I am quite busy and screenshots are a bit of a low hanging fruits so...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants