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

Icon overhaul #213

Merged
merged 1 commit into from
Aug 28, 2022
Merged

Icon overhaul #213

merged 1 commit into from
Aug 28, 2022

Conversation

Ch1llb0
Copy link
Contributor

@Ch1llb0 Ch1llb0 commented Aug 28, 2022

All icons were reworked:

  • channel icons turned monochrome mimicing the top corner icon during live TV broadcasting
  • streamline non-channel icon style
  • make all icons feature high resolution
  • minor fixes (renaming icons, adding missing ones)

Should address #212 and #194

- turn channel icons monochrome (live TV top corner icon style)
- turn all icons high res
- streamline non-channel icons to be same style
@Ch1llb0
Copy link
Contributor Author

Ch1llb0 commented Aug 28, 2022

This is how things will look with Estuary:

screenshot00006
screenshot00007
screenshot00008
screenshot00009
screenshot00010
screenshot00011
screenshot00012

And with a skin like the OSMC Skin that has different visibility needs when it comes to icon and fanart colours:

screenshot00005
screenshot00000
screenshot00001
screenshot00002
screenshot00003
screenshot00004

@codingPF codingPF changed the base branch from master to icons August 28, 2022 20:29
@codingPF codingPF merged commit 9aa3c69 into mediathekview:icons Aug 28, 2022
@Ch1llb0 Ch1llb0 deleted the IconOverhaul branch August 28, 2022 20:38
@Ch1llb0
Copy link
Contributor Author

Ch1llb0 commented Aug 29, 2022

Die kleinen Icons könnten eventuell noch einen kleinen transparenten Rahmen für Estuary gebrauchen... Soll ich das noch ergänzen?

Und rein vom Design her: Die Zahlen ausgefüllt weiß oder wie momentan lediglich umrandet?

@codingPF
Copy link
Member

Mit Zahlen meinst du die Letter?
Finde das so auch nett (als Ballonschrift) aber der Rahmen fehlt damit es nicht so geknautscht Aussicht…

@Ch1llb0
Copy link
Contributor Author

Ch1llb0 commented Aug 29, 2022

OK, dann bleiben die Lettern so. Die Rahmen ergänze ich noch und öffne einen neuen PR

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.

2 participants