Skip to content

flatpak: Update images for Flathub#878

Open
Venefilyn wants to merge 1 commit into
cockpit-project:mainfrom
Venefilyn:flatpak/image-update
Open

flatpak: Update images for Flathub#878
Venefilyn wants to merge 1 commit into
cockpit-project:mainfrom
Venefilyn:flatpak/image-update

Conversation

@Venefilyn

@Venefilyn Venefilyn commented Mar 23, 2026

Copy link
Copy Markdown
Member

Current images are outdated and show older design. This updates the
images from my machine running latest stable Cockpit Client on
localhost.

Caveat of this is that overview shows

System manufacturer System Product name

Which is strange but ultimately something my machine reports, so no bug
within Cockpit. However, might raise eyebrows for those looking at the
images.

Images are found in
https://flathub.org/en/apps/org.cockpit_project.CockpitClient

Signed-off-by: Freya Gustavsson freya@venefilyn.se

@allisonkarlitskaya allisonkarlitskaya left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

https://docs.flathub.org/docs/for-app-authors/metainfo-guidelines/quality-guidelines#reasonable-window-size mentions

In order to ensure that text and interface elements are visible scaled down in the app store UI the window size should be 1000x700 pixels or smaller (2000x1400 for HiDPI).

... without defining what HiDPI means or how we might indicate that our images are HiDPI.

These new screenshots are more like 3000x2000, so they need to be scaled down some. In general, I guess the idea is that we should make the window "smaller" in the sense of having a higher ratio of content to padding, with fonts as large as possible (it mentions scaling down) without scrollbars.

It would also be good to look into how we define if these are HiDPI and what the scaling factor was...

@Venefilyn

Copy link
Copy Markdown
Member Author

@allisonkarlitskaya I'm struggling to get an answer from Flathub about what they mean with HiDPI and what the scaling factor would be etc.

I have a 4K monitor with 166% scaling so that's why the images are big. Are you able to take smaller images then? I scaled down almost as much as I could in the images I took so that there are no scrollbars while all images being the same size

@jelly

jelly commented Mar 27, 2026

Copy link
Copy Markdown
Member

Small nit, I think it would be better if we took screenshots of Fedora and not Bazzite :-)

A virtual machine gives me "Model" QEMU Standard PC (i440FX + PIIX, 1996) which is a bit nicer then the standard bios info. Also you might consider not publishing your machine-id as that's unique per machine.

@razzeee

razzeee commented Mar 27, 2026

Copy link
Copy Markdown

The images should be smaller, even for hdpi

@Venefilyn

Copy link
Copy Markdown
Member Author

The images should be smaller, even for hdpi

@razzeee what is HiDPI supposed to be? It isn't super obvious from the docs what relative sizing it is or something like that :D

@razzeee

razzeee commented Mar 27, 2026

Copy link
Copy Markdown

@Venefilyn

Copy link
Copy Markdown
Member Author

@allisonkarlitskaya okay so either we can dismiss the reasonable window size and just go for smallest possible size without introducing scrollbars, cause otherwise it's just impossible

Here is the 1x size of Cockpit Client at 1000x700
Screenshot From 2026-04-14 17-41-27
Screenshot From 2026-04-14 17-40-55
Screenshot From 2026-04-14 17-41-45

And 2x/HiDPI size 2000x1400
Screenshot From 2026-04-14 17-42-36
Screenshot From 2026-04-14 17-42-18
Screenshot From 2026-04-14 17-42-06

I'd vote for going for 2x, setting smallest res which would be 2496x1802, or if I were to take it without HiDPI it would be 1248x901.
image

@Venefilyn

Venefilyn commented Apr 14, 2026

Copy link
Copy Markdown
Member Author

@allisonkarlitskaya We could also go with the default requested size we get for Cockpit Client

image

@allisonkarlitskaya

Copy link
Copy Markdown
Member

They all seem kinda fine to me. Let's talk tomorrow?

Current images are outdated and show older design. This updates the
images from my machine running latest stable Cockpit Client on
a Fedora QEMU VM.

Signed-off-by: Freya Gustavsson <freya@venefilyn.se>
@Venefilyn Venefilyn force-pushed the flatpak/image-update branch from 1cda6a4 to 9e25e17 Compare June 24, 2026 14:48

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Another one just to show dark theme. Can be added to the metadata file

@Venefilyn

Copy link
Copy Markdown
Member Author

@allisonkarlitskaya Images updated with white theme, 1x sizing, and as small as I can without introducing scrollbars. PTAL.

Also added one dark-theme image which we'll add to the metadata file

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.

4 participants