Compare Two mode loads wrong post for left side when another post was created at the same time

[westi]

In the revisions js we have the following code:

'compareTo': self.revisions.at( self.leftDiff ).get( 'ID' ) - 1,

Performing arbitrary increments or decrements on the ID won't work because there is no guarantee that they will be sequential on anything.

I'm attaching a patch that appears to fix it for me, but I'm not 100% convinced of what the logic is here - the JS could use some comments so it is clear what is going on.

[westi]

Simple patch which seems to fix the issue.

[ocean90]

Seems like just a typo.

[ocean90]

For the right side it's correct, see trunk/wp-admin/js/revisions.js:24020#L184​.

[ocean90]

In 24040:

Revisions JS: Fix typo. props westi . fixes #24136.

[adamsilverstein]

i believe values from the slider are off by one compared to the model index, hence the -1;

i will work to add better inline documentation :)

[adamsilverstein]

i added some inline documentation in #23897;

also did a little testing and verified the origin of all the -1 you see in the code - the diff values are indeed one off the model index.

jQuery UI slider reports leftmost position as 1 for two handled mode and 0 for one handled mode, the revisions code assumes left most position is 1 and behaves accordingly; since single handled mode returns left most as 0, it actually ADDS 1 to that value to keep it consistent with the two handled position.

it might be possible to change the stored value in the slider callbacks instead, but i'm nervous about changing/breaking what already works.

[ocean90]

In 24116:

Optimize jQuery selector for checkbox synchronization. props SergeyBiryukov. see #24136.

[ocean90]

[24116] was for #22975.