ajayyy / DeArrow

Crowdsourcing better titles and thumbnails on YouTube
https://dearrow.ajay.app
GNU General Public License v3.0
1.35k stars 39 forks source link

Don't replace title if it is only formatted differently when title formatting disabled #145

Closed MrJPGames closed 12 months ago

MrJPGames commented 12 months ago

The issue is easiest explained through this example: https://www.youtube.com/watch?v=5cwdhuWqY0Q The original title of that video is: "Two Grant Imahara Stories" DeArrow user submitted version: "Two grant imahara stories" (ignoring the fact the name is incorrectly not capitalized here).

The problem is that the submitted title in this example should never be used. For users that prefer this title format the "First letter upercase" "Title Format" setting should be used instead. For other users (such as myself) that dislike this formatting it should not be imposed because someone erroneously submitted their personal formatting preference.

I'd suggest that locally the current DeArrow title is compared to the current official title. If any of the "Title Format" options result in the DeArrow title it should be ignored, and the original title + the user's selected formatting should be used instead. Alternatively you might want to implement a check for alternative titles and find the best rated one that is not the original title formatted differently, and use that instead, if possible and only if not fallback to the original.

EDIT: This should likely also check for emoji removal etc. In other words any time it's relatively easy to check if a user submission could've been created using a built-in DeArrow setting that title should be rejected. Users preferring that style should use the setting and submissions should only be used in cases where the content not the style of the title has to change.

ajayyy commented 12 months ago

If there is a format selected, then we do want to allow this as the use-case is correcting issues with the title-formatter and making sure it knows which words are proper nouns.

But, if title formatting is disabled, then it probably should be ignored.

MrJPGames commented 12 months ago

Edit: missed closed due to commit, read it as just closed. Thanks a lot for adding the feature/change!