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

[Proposal] Screenshots #4

Merged
merged 7 commits into from
Oct 24, 2022
Merged

[Proposal] Screenshots #4

merged 7 commits into from
Oct 24, 2022

Conversation

Ilithy
Copy link
Contributor

@Ilithy Ilithy commented Oct 24, 2022

  • (first drafts) Added screenshots of the application.
  • Added links in the readme to the screenshots
  • Fixed the "get it on Github" badge
  • Added IzzyOnDroid link and badge
  • Many thanks from me for this new app, which has perfectly found its place on my pixel, and which already works very well ❤️

@Bnyro
Copy link
Member

Bnyro commented Oct 24, 2022

Thank you, highly appreciated <3
However, the GitHub link seems to use a F-Droid logo instead of the Github badge :)

@Bnyro Bnyro merged commit 6a6fd6c into you-apps:main Oct 24, 2022
@Bnyro
Copy link
Member

Bnyro commented Oct 24, 2022

cc @IzzySoft
Thanks to @Ilithy we got screenshots now, would be great if you could include them into your repo as well now :)
Thank you!

@IzzySoft
Copy link

Updated the config and manually triggered an update – thanks! Would be better if they came without that framing (apart of transparency getting lost when converting to JPG, it wastes useful screen estate on small screens – and yes, I convert to JPG as it reduces size by 50%+ which is good for low-end devices and small data plans, plus for faster page loads 😉)

@Bnyro
Copy link
Member

Bnyro commented Oct 24, 2022

Updated the config and manually triggered an update – thanks!

Great, thank you for fetching them <3

Would be better if they came without that framing (apart of transparency getting lost when converting to JPG, it wastes useful screen estate on small screens – and yes, I convert to JPG as it reduces size by 50%+ which is good for low-end devices and small data plans, plus for faster page loads 😉)

Fair, we'll use JPGs without the framing next time to impress with great screenshots :)

@IzzySoft
Copy link

Wonderful, thanks!

@Ilithy
Copy link
Contributor Author

Ilithy commented Oct 24, 2022

@IzzySoft and @Bnyro
Sorry, I don't normally have these constraints, so I don't really adjust to the lightest possible, just enough so that it's not too much ;-);

I'll try to make it lighter from now on (100K max per image seems acceptable?).

Should I remove the transparent background too? That looks cleaner (I think), whether the user is using a light or dark theme, it fits better...(but I can basically not add it) can I leave it on if I can stay under 100KB per image?

For the frame, you decide, should I make basic captures? (This is said without malice)

Thanks

@IzzySoft
Copy link

No prob, @Ilithy – one cannot be aware of everything. I usually try to keep screenshots as light as possible (they should give an impression of the app, they're not a "high-res image gallery" 🙈). So I resize to 350:x (portrait) and hope they get below 30k each (on average), which in most cases involves converting them to JPG. Which also means: no prob if yours are at "full HD", my updater takes care for that.

Should I remove the transparent background too?

Makes sense, as converting to JPG makes it greenish; if you remove the frames anyway, it would be gone anyway, right? And yes, "basic captures" would be great! As pointed out, users whould get an impression of what to expect. You can still have a high-res image gallery elsewhere. If e.g. you have a nice Wiki page (Github offers that feature, so do Codeberg and GitLab should you move one day), I could configure that as "app website" to point users there.

That all are just suggestions, I of course take what's there. Just for best results… well, you get the idea 😉

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.

3 participants