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

Change "not available" image from PNG to vector format #6917

Merged

Conversation

sherlockbeard
Copy link
Contributor

changed the not available .png to a xml file

Fixes #5609

fixed the issue of large devices . extension of #5925

What is it?

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

photo1628868390

tablet view
Screenshot_1628868849

@Stypox
Copy link
Member

Stypox commented Aug 14, 2021

I tested on API 19 (Android 4.4) and it works only on white theme. Otherwise a gray or black background is just shown.

@sherlockbeard
Copy link
Contributor Author

@Stypox i just tested on virtual machine android version 4.4 api 19. i was not able to reproduce your error .

untitled

@triallax triallax changed the title Sherlockbeard not available vector Make "not available" image from PNG to vector format Aug 14, 2021
@triallax triallax changed the title Make "not available" image from PNG to vector format Change "not available" image from PNG to vector format Aug 14, 2021
@triallax triallax added the feature request Issue is related to a feature in the app label Aug 14, 2021
@nadiration
Copy link
Contributor

@sherlockbeard thank you for reconsidering this. No more boring grey background

@sherlockbeard
Copy link
Contributor Author

@Stypox can you give me more information about the issue and the device you are testing with

@Stypox
Copy link
Member

Stypox commented Aug 16, 2021

@sherlockbeard I was using the emulator with 240x320:ldpi screen and Android 4.4 x86 image. But now I can't reproduce anymore.

@sherlockbeard
Copy link
Contributor Author

@Stypox . Ok so you are not able to reproduce so should I consider the issue (the one you mentioned in the comment) fixed?

Copy link
Member

@Stypox Stypox left a comment

Choose a reason for hiding this comment

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

Yes, consider it fixed. Thank you!

@tsiflimagas
Copy link
Contributor

I guess this can be merged(?)

@Stypox
Copy link
Member

Stypox commented Aug 25, 2021

@tsiflimagas yes, I forgot about it, thank you :-)

@Stypox Stypox merged commit bbcfdf2 into TeamNewPipe:dev Aug 25, 2021
This was referenced Sep 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Issue is related to a feature in the app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The "Video not available" thumbnail on unplayable videos needs some changes
5 participants