[Bug 25294] Reject button confirmation screen in Pending Changes

2010-10-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

Chad H. innocentkil...@gmail.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #11 from Chad H. innocentkil...@gmail.com 2010-10-26 02:37:51 UTC 
---
Committed in r75408 with one last minor fix.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-10-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

Chad H. innocentkil...@gmail.com changed:

   What|Removed |Added

   Attachment #7741|0   |1
is obsolete||

--- Comment #10 from Chad H. innocentkil...@gmail.com 2010-10-24 15:01:21 UTC 
---
Created attachment 7753
  -- https://bugzilla.wikimedia.org/attachment.cgi?id=7753
v2 of reject button

Incorporated Aaron's feedback. Now fully functional, but ugly :)

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-10-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

--- Comment #9 from Aaron Schulz jschulz_4...@msn.com 2010-10-21 18:41:52 UTC 
---
A few things I noticed:
i) Code is missing $form .= Html::closeElement( 'form' );
ii) Also missing $form .= Html::hidden( 'target',
$oldRev-getTitle()-getPrefixedDBKey() );
iii) Also missing $form .= Html::hidden( 'wpEditToken', $wgUser-editToken()
);
iv) I assume that $wgUser is just there for the quick version (the class field
user might be better, or a User argument)

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-10-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

--- Comment #8 from Chad H. innocentkil...@gmail.com 2010-10-20 07:21:20 UTC 
---
Created attachment 7741
  -- https://bugzilla.wikimedia.org/attachment.cgi?id=7741
v1 of reject confirmation

Work-in-progress on confirmation button. Currently brings you to the
confirmation page, displays the information, and then goes nowhere when
pressing OK.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-10-20 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

Chad H. innocentkil...@gmail.com changed:

   What|Removed |Added

   Keywords||need-review, patch

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-09-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

Rob Lanphier ro...@wikimedia.org changed:

   What|Removed |Added

 AssignedTo|ro...@wikimedia.org |innocentkil...@gmail.com

--- Comment #2 from Rob Lanphier ro...@wikimedia.org 2010-09-27 19:31:22 UTC 
---
(In reply to comment #1)
 It doesn't make sense, if you introduce confirmation screen, there will 2 the
 same  buttons:
 * Reject,
 * Undo.
 
 Currently Reject button is very useful - it is improved version of 
 Rollback
 button (you can automatically revert changes of all authors, not only of one
 author).

There is a lot of resistance to creating something that works like rollback,
especially because of the multi-author nature of it.  The fear is that good
edits will get lumped in with the bad, and thus we'll lose edits and alienate
new editors.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-09-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

--- Comment #3 from Leinad danny.lei...@gmail.com 2010-09-27 20:30:45 UTC ---
If you do not want create something that works like rollback, I do not
understand why you created Reject... you have Undo with the same
funcionality.

So I will try to propose a compromise solution:
Could you create two new permissions relating to Reject button:
* 'soft-reject' - similar to Undo,
* 'hard-reject' - similar to Rollback.

And 'hard-reject' permission should be assigned by default for 'sysop' group
(the same as 'rollback' permission).

Plus communities will be able to request to enable 'soft-reject' or
'hard-reject' permission for 'editor' group.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-09-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

--- Comment #4 from Rob Lanphier ro...@wikimedia.org 2010-09-27 20:44:58 UTC 
---
(In reply to comment #3)
 So I will try to propose a compromise solution:
 Could you create two new permissions relating to Reject button:
 * 'soft-reject' - similar to Undo,
 * 'hard-reject' - similar to Rollback.

Maybe in the future, but not for the November release.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-09-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

--- Comment #5 from Chad H. innocentkil...@gmail.com 2010-09-27 20:59:49 UTC 
---
(In reply to comment #3)
 So I will try to propose a compromise solution:
 Could you create two new permissions relating to Reject button:
 * 'soft-reject' - similar to Undo,
 * 'hard-reject' - similar to Rollback.
 

No, we do not need more permissions for this. Nor should it act like rollback
and be automatic. Since you are potentially affecting multiple users, a
confirmation should be required.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-09-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

--- Comment #6 from Leinad danny.lei...@gmail.com 2010-09-27 21:14:17 UTC ---
(In reply to comment #5)
 No, we do not need more permissions for this. Nor should it act like rollback
 and be automatic. Since you are potentially affecting multiple users, a
 confirmation should be required.

I proposed *compromise solution*. 'hard-reject' would not be the default for
all projects.

For example on Polish Wikipedia all editors have 'rollback' permission and a
lot of people complaining that can't revert all unsighted changes using
rollback.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-09-27 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

--- Comment #7 from Rob Lanphier ro...@wikimedia.org 2010-09-27 21:36:01 UTC 
---
Hi Leinad, I know you're trying to reach a compromise.  It sounds like the
Polish Wikipedia community has very different expectations surrounding how this
feature should work than the English Wikipedia community does.

As I said before, we really don't want to make this an option for the November
release.  The reason is that I believe it's going to take an actual deployment
of the feature to have an informed and inclusive conversation about what
options should exist for the feature.

One thing we have tentatively planned after November is a fork of this
extension into two:  a Pending Changes variant where we strip away a lot of
the complexity in the current codebase, and a FlaggedRevs version which is a
continuation of the existing version.  Once we do that, we'll be in a better
place to actually talk about making this optional.

That said, I'm not sure making it optional is really the right thing to do. 
One reason why the code we have is so complicated is because we have so many
options.  Hence, there still will probably be resistance after November to
adding a new option, but it may be the case that the FlaggedRevs fork can
actually carry the default that you would like to see.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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 25294] Reject button confirmation screen in Pending Changes

2010-09-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25294

Leinad danny.lei...@gmail.com changed:

   What|Removed |Added

 CC||danny.lei...@gmail.com

--- Comment #1 from Leinad danny.lei...@gmail.com 2010-09-25 13:49:20 UTC ---
It doesn't make sense, if you introduce confirmation screen, there will 2 the
same  buttons:
* Reject,
* Undo.

Currently Reject button is very useful - it is improved version of Rollback
button (you can automatically revert changes of all authors, not only of one
author).

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
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