[Bug 63442] [[Special:Diff]] - counterintuitive order of oldid/newid parameters
https://bugzilla.wikimedia.org/show_bug.cgi?id=63442 --- Comment #5 from Jérémie Roquet jroq...@arkanosis.net --- Hi everyone, This is just my opinion, but I don't think the current behavior is less intuitive than the suggested one — rather the opposite, actually: - the left parameter is what will be shown on the left side of your screen and the right parameter on the right side (remember that it's possible to diff revisions in whatever order you want; it'll just display the diff leftside right); - this syntax is consistent with Diff/oldid/next and Diff/oldid/cur (arguably, it's not with Diff/newid/prev, though); - this is similar to the syntax of popular VCS, such as subversion (svn diff -c newid vs svn diff -r oldid:newid) and git (git show newid vs git diff oldid:newid). If anything is to be changed, for me it would be to make “prev”, “next” and “cur” usable both as first and second parameters (with the result of Diff/newid/prev syntax meaning the opposite of what it means today, ie. newid shown on the left and prev shown on the right). Best regards, -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63442] [[Special:Diff]] - counterintuitive order of oldid/newid parameters
https://bugzilla.wikimedia.org/show_bug.cgi?id=63442 --- Comment #6 from Eduard Braun eduard.bra...@gmx.de --- (In reply to Jérémie Roquet from comment #5) If anything is to be changed, for me it would be to make “prev”, “next” and “cur” usable both as first and second parameters (with the result of Diff/newid/prev syntax meaning the opposite of what it means today, ie. newid shown on the left and prev shown on the right). Actually (from my side) this would be fine, too. For me the crucial point is consistency. That is meaning (or order) of the parameters shouldn't change regardless of what kind of parameters you pass (an ID or prev/cur/next). Therefore it basically breaks down to these two options: 1) [[Special:Diff/right_version/left_version]] 2) [[Special:Diff/left_version/right_version]] If only one parameter is specified both cases above would get [[Special:Diff/right_version]] 1) is what I initially proposed. It's advantage is that also if only one parameter is specified, it still is the version on the right in the current implementation. Therefore the meaning of the first parameter *never* changes. 2) is slightly more intuitive in that it normally orders versions chronologically (since normally the older version will be shown on the left and the newer version will be shown on the right). However if only one parameter is specified, this will be the version that is shown on the right therefore switching the meaning of first/second parameter. All in all I could live with both options, but changing behavior based on whether an ID is given or a relation like prev/cur/next is inconsistent and seems wrong to me. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63442] [[Special:Diff]] - counterintuitive order of oldid/newid parameters
https://bugzilla.wikimedia.org/show_bug.cgi?id=63442 Andre Klapper aklap...@wikimedia.org changed: What|Removed |Added Priority|Unprioritized |Lowest Version|unspecified |1.23-git Summary|[[Special:Diff]] - |[[Special:Diff]] - |counterintuitive order of |counterintuitive order of |parameters |oldid/newid parameters --- Comment #1 from Andre Klapper aklap...@wikimedia.org --- Any idea how long it's been like this? Wondering if breaking is really worth it... -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63442] [[Special:Diff]] - counterintuitive order of oldid/newid parameters
https://bugzilla.wikimedia.org/show_bug.cgi?id=63442 Eduard Braun eduard.bra...@gmx.de changed: What|Removed |Added CC||eduard.bra...@gmx.de --- Comment #2 from Eduard Braun eduard.bra...@gmx.de --- This has been merged on January 26th, see https://gerrit.wikimedia.org/r/#/c/63395/ I think usage (with two parameters) is still very low so we wouln't break much if we make the change now. If you look at the linked commit, the actual behaviour is even contrary to the one implied by the commit message (which uses 12345 as the diff ID in the first four examples therfore suggesting 98765 would be the oldid in the last example). So it could even be considered a bug and for not working as designed. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63442] [[Special:Diff]] - counterintuitive order of oldid/newid parameters
https://bugzilla.wikimedia.org/show_bug.cgi?id=63442 Eduard Braun eduard.bra...@gmx.de changed: What|Removed |Added CC||jroq...@arkanosis.net, ||matma@gmail.com --- Comment #3 from Eduard Braun eduard.bra...@gmx.de --- CC'ing authors of the commit so they can add their judgment. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 63442] [[Special:Diff]] - counterintuitive order of oldid/newid parameters
https://bugzilla.wikimedia.org/show_bug.cgi?id=63442 --- Comment #4 from Bartosz Dziewoński matma@gmail.com --- This is the expected behavior (you're right that the commit message is not clear on this) and personally I think this order is more natural (even if inconsistent with the single-parameter version), but I'm not particularly attached to it. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
