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 "What's New" icon #9678

Merged
merged 2 commits into from
Jan 15, 2023
Merged

Conversation

Marius1501
Copy link
Contributor

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

  • Changed the What's New icon
  • Now it has a Youtube-styling

Before/After Screenshots/Screen Record

NewPipe2 – ic_whats_new xml  NewPipe2 app main  13 01 2023 16_37_10

Fixes the following issue(s)

Due diligence

@@ -0,0 +1,10 @@
<vector xmlns:android="http://schemas.android.com/apk/res/android"
Copy link
Member

Choose a reason for hiding this comment

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

This icon is called subscriptions here, so please keep the original name for consistency

@tvnmguy
Copy link

tvnmguy commented Jan 13, 2023

There is some changes needs;

[1]
IMG_20230114_035633.jpg
[2]
IMG_20230114_035449.jpg
[3]
IMG_20230114_040437.jpg
[4]
IMG_20230114_040832.jpg

@Marius1501
Copy link
Contributor Author

@tvnmguy all the changes you mentioned are already in this PR except for the naming of the What's New and the Subscriptions section. Should we really rename these sections?

@sonarcloud
Copy link

sonarcloud bot commented Jan 14, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@Stypox
Copy link
Member

Stypox commented Jan 15, 2023

Should we really rename these sections?

No, "what's new" is a perfectly valid name. "Subscriptions" and "Channels" might be confused instead.

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.

Works, thank you!

@Stypox Stypox merged commit 8699779 into TeamNewPipe:dev Jan 15, 2023
@Marius1501 Marius1501 deleted the change_whats_new_icon branch January 15, 2023 14:48
@Stypox Stypox mentioned this pull request Jan 22, 2023
3 tasks
@AudricV AudricV changed the title Changed the What's New icon Change "What's New" icon Feb 28, 2023
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.

Change the icon of subscription update
3 participants