Skip to content

Conversation

@danirabbit
Copy link
Member

@danirabbit danirabbit commented Mar 6, 2025

Screenshot from 2025-09-05 08 18 14

@danirabbit

This comment was marked as outdated.

@danirabbit danirabbit added this to OS 9 Aug 22, 2025
@danirabbit danirabbit moved this to In progress in OS 9 Aug 22, 2025
@danirabbit danirabbit requested a review from a team August 30, 2025 17:08
@danirabbit danirabbit marked this pull request as ready for review August 30, 2025 17:08
@danirabbit danirabbit moved this from In progress to Needs Review in OS 9 Aug 30, 2025
Copy link
Contributor

@jeremypw jeremypw left a comment

Choose a reason for hiding this comment

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

The screenshot is often not the correct one. Also makes list visually jumpy.

@danirabbit
Copy link
Member Author

danirabbit commented Sep 8, 2025

@jeremypw can you elaborate on what you mean by "the correct one"? Are there certain apps you're reproducing a problem with?

For the jumpiness, we could always preserve the height and have a image missing placeholder instead of showing nothing 🤔

@jeremypw
Copy link
Contributor

jeremypw commented Sep 9, 2025

@danirabbit I ran the search "term" and I noticed that many (50%?) of the screenshots did not correspond to any of the screenshots shown in the app details page and appeared to be of a different app entirely. For example the "Blackbox" app.

Screenshot from 2025-09-09 10 27 37

@danirabbit danirabbit removed this from OS 9 Dec 7, 2025
@danirabbit danirabbit moved this to In progress in OS 8.1.1 Dec 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

5 participants