Skip to content

Fix "Get it on F-Droid" appearing giant in README#12409

Merged
Stypox merged 2 commits intoTeamNewPipe:devfrom
Stypox:readme-badge-size-fix
Jul 7, 2025
Merged

Fix "Get it on F-Droid" appearing giant in README#12409
Stypox merged 2 commits intoTeamNewPipe:devfrom
Stypox:readme-badge-size-fix

Conversation

@Stypox
Copy link
Member

@Stypox Stypox commented Jul 6, 2025

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

I updated the message about the rewrite at the top of the README, because it was outdated. I did so only in the English README though.

I also fixed the size of "Get it on F-Droid", because GitHub somehow changed how images are rendered. This is how is used to look (taken from wayback machine):

image

This is how it looks now:

image

And this is how this PR fixes it (the image is 646x250, and the previous height was 80, so the corresponding width is ~206):

image

Due diligence

@github-actions github-actions bot added the size/small PRs with less than 50 changed lines label Jul 6, 2025
@Stypox Stypox added meta Related to the project but not strictly to code localisation / translation Everything that has to do with translations or Weblate labels Jul 6, 2025
@Stypox Stypox force-pushed the readme-badge-size-fix branch from 22bec75 to 5960305 Compare July 7, 2025 14:24
@Stypox Stypox force-pushed the readme-badge-size-fix branch from 5960305 to f0b26e2 Compare July 7, 2025 14:25
@Stypox Stypox merged commit 8ab7948 into TeamNewPipe:dev Jul 7, 2025
1 check passed
@Stypox Stypox deleted the readme-badge-size-fix branch July 16, 2025 13:53
@Stypox Stypox mentioned this pull request Jul 17, 2025
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

localisation / translation Everything that has to do with translations or Weblate meta Related to the project but not strictly to code size/small PRs with less than 50 changed lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants