Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Format dislikes the same way likes are formatted #202

Closed
wants to merge 3 commits into from
Closed

Format dislikes the same way likes are formatted #202

wants to merge 3 commits into from

Conversation

oSumAtrIX
Copy link
Contributor

@oSumAtrIX oSumAtrIX commented Dec 5, 2021

Change function numberFormat to match the formatting on YouTube more closely.
In the range of millions, YouTube hides decimals, which numberFormat does not do yet.
Related issue: #201

Copy link

@CutestNekoAqua CutestNekoAqua left a comment

Choose a reason for hiding this comment

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

overall good?

});

return formatter.format(roundDown(numberState)).replace(/\.0|,0/, "");
function numberFormat(num) {

Choose a reason for hiding this comment

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

maybe let the user choose what way they want it displayed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For the extension a toggle can be implemented into the pop-out, though for tampermonkey users another solution has be to created

@Anarios
Copy link
Owner

Anarios commented Dec 5, 2021

Won't work for international users, locale has to be used

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants