-
Notifications
You must be signed in to change notification settings - Fork 362
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
Switch bit weights between Special and Speed stats #194
Conversation
js/shared_controls.js
Outdated
@@ -85,8 +85,8 @@ $(".sl .dvs").keyup(function () { | |||
function getHPDVs(poke) { | |||
return (~~poke.find(".at .dvs").val() % 2) * 8 + | |||
(~~poke.find(".df .dvs").val() % 2) * 4 + | |||
(~~poke.find(gen === 1 ? ".sl .dvs" : ".sa .dvs").val() % 2) * 2 + | |||
(~~poke.find(".sp .dvs").val() % 2); | |||
(~~poke.find(".sp .dvs").val() % 2) * 2 + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove white space
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe good enough if I put them all in one line, so it doesn't end up like the first recovery text PR (messy indenting everywhere) ?
Hmm guess it’s not so easy on mobile, if you can’t fix it, I’ll fix it for you when I test it. You should be able to highlight white space with your mouse highlighter in case you weren’t aware. |
Looks way better now. |
So, what now ? |
Gotta fix the spacing for this and I’m going to give sadisticmystic a bit more time to verify his issue is fixed |
That works. |
ugh annoying, but fixed lol. Also thanks for the confirmation @SadisticMystic |
Closes issue #182.. I guess.
In short : instead of considering 2 bits for Special and one for Speed, the Speed DVs affect 2 units of HP DVs, while the Special ones do not affect but one.