Skip to content

Fix the theming of the share menu logos for the system, black and dracula themes#2330

Merged
PrestonN merged 1 commit intoFreeTubeApp:v0.17.0-RCfrom
absidue:fix-share-menu-logos-rc
Jun 20, 2022
Merged

Fix the theming of the share menu logos for the system, black and dracula themes#2330
PrestonN merged 1 commit intoFreeTubeApp:v0.17.0-RCfrom
absidue:fix-share-menu-logos-rc

Conversation

@absidue
Copy link
Member

@absidue absidue commented Jun 17, 2022


Fix the theming of the share menu logos for the system, black and dracula themes (RC branch)

Pull Request Type

  • Bugfix

Description
This pull request fixes the YouTube and Invidious logos being invisible in the share menu when the system, black or dracula themes are active.

Screenshots (if appropriate)

System

before

after

Black

before

after

Dracula

before

after

Testing (for code that is not small enough to be easily understandable)
I tested this pull request by changing the theme and checking that the logos are visible regardless of the selected theme.

Desktop (please complete the following information):

  • OS: Windows
  • OS Version: 10
  • FreeTube version: c12d64c

@absidue absidue added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 17, 2022
@PrestonN PrestonN enabled auto-merge (squash) June 17, 2022 11:49
PikachuEXE
PikachuEXE previously approved these changes Jun 17, 2022
@absidue absidue changed the base branch from v0.17.0-RC to development June 19, 2022 18:54
@absidue absidue dismissed stale reviews from efb4f5ff-1298-471a-8973-3d47447115dc and PikachuEXE June 19, 2022 18:54

The base branch was changed.

@absidue absidue changed the base branch from development to v0.17.0-RC June 19, 2022 18:55
@absidue absidue closed this Jun 19, 2022
auto-merge was automatically disabled June 19, 2022 19:34

Pull request was closed

@absidue absidue reopened this Jun 19, 2022
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 19, 2022
@PrestonN PrestonN enabled auto-merge (squash) June 19, 2022 19:34
@absidue absidue added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 19, 2022
@PrestonN PrestonN merged commit 21f3862 into FreeTubeApp:v0.17.0-RC Jun 20, 2022
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Jun 20, 2022
@absidue absidue deleted the fix-share-menu-logos-rc branch June 20, 2022 05:46
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.

5 participants