Skip to content
This repository has been archived by the owner on Jul 18, 2024. It is now read-only.

youtube-cleanup: remove outdated rules targeting old design #294

Merged
merged 4 commits into from
Jan 8, 2023

Conversation

JohnyP36
Copy link
Member

@JohnyP36 JohnyP36 commented Nov 3, 2022

@JohnyP36 JohnyP36 requested a review from xvello as a code owner November 3, 2022 16:38
@JohnyP36 JohnyP36 changed the title Removed redundant/old rules Removed redundant/old rules on YT Nov 3, 2022
@Nomes77
Copy link
Member

Nomes77 commented Nov 3, 2022

##.ytd-watch-metadata > .top-level-buttons #text:not(:has-text(/[\d]/)) is the second newest design
and the same for ##[class*="video-secondary-info-renderer"] > #items:has(.ytd-video-description-music-section-renderer).
It could be possible that not everyone has the newest design.
Thus, it would be save to keep that rule for a moment.

You made a duplicate of #293 so I hope you don't mind I'm closing this PR

@Nomes77 Nomes77 closed this Nov 3, 2022
@JohnyP36
Copy link
Member Author

JohnyP36 commented Nov 3, 2022

##.ytd-watch-metadata > .top-level-buttons #text:not(:has-text(/[\d]/)) is the second newest design. If could be possible that not everyone has the newest design. Thus, it would be save to keep that rule for a moment.

You made a duplicate of #293 so I hope you don't mind I'm closing this PR

He asked me to do it. NOT you.

@Nomes77
Copy link
Member

Nomes77 commented Nov 3, 2022

@JohnyP36 I'm sorry, but I saw his question. Plus I removed more from the old-old YoutTube design.

@xvello xvello reopened this Nov 3, 2022
@xvello
Copy link
Member

xvello commented Nov 3, 2022

Heya @JohnyP36 and @BPower0036 . Thank you both for your quick response! It's unfortunate that you duplicated work, and I'm sorry about that!

I do agree that it's probably better to keep the "old-new" rules for now, as I'm not sure that the new layout has been rolled out to all countries. Checking the git blame for this file should help us identify what rules are the oldest and should be removed. Could we focus this PR on the youtube-cleanup template only, and discuss the other templates in #293?

@JohnyP36
Copy link
Member Author

JohnyP36 commented Nov 3, 2022

Heya @JohnyP36 and @BPower0036 . Thank you both for your quick response! It's unfortunate that you duplicated work, and I'm sorry about that!

I do agree that it's probably better to keep the "old-new" rules for now, as I'm not sure that the new layout has been rolled out to all countries. Checking the git blame for this file should help us identify what rules are the oldest and should be removed. Could we focus this PR on the youtube-cleanup template only, and discuss the other templates in #293?

Yes, that's fine.
Before removing the old rules, I indeed check the git blame, but one is from 5 months ago and the other from 4.
Up until a week ago I always had the layout from 5 months ago (both when I was logged in to Youtube and when logged out).
To me it seems that both the 4-and-5-months-ago layouts have been replaced with the new one -- with rounded corners.


Edit
5 months ago = May 24, 2022
4 months ago = July 7, 2022

The old rules regarding remove-text-after-buttons-below-video are from May 19, 2022

@github-actions
Copy link
Contributor

Thanks for your contribution! The automated tests passed, we will review your PR shortly!

@JohnyP36
Copy link
Member Author

JohnyP36 commented Jan 5, 2023

@xvello I think it's save to merge this PR now without disturbing users that are using those rules, as it seems that the new YT layout has been rolled out to every country.

@xvello xvello changed the title Removed redundant/old rules on YT youtube-cleanup: remove outdated rules targeting old design Jan 8, 2023
Copy link
Member

@xvello xvello left a comment

Choose a reason for hiding this comment

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

Thanks @JohnyP36! Sorry for the low responsiveness, life is very busy right now, but it should hopefully settle in a week or two.

@xvello xvello merged commit 6aca06a into letsblockit:main Jan 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

3 participants