Page MenuHomePhabricator

$wgThanksConfirmationRequired should be a preference
Closed, DeclinedPublic

Description

Users should have a preference to control whether they wish to confirm sending thanks, rather than having $wgThanksConfirmationRequired control it for all users.


Version: unspecified
Severity: enhancement
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=53879

Details

Reference
bz59807

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 2:28 AM
bzimport added a project: Thanks.
bzimport set Reference to bz59807.
bzimport added a subscriber: Unknown Object (MLST).

swalling wrote:

What if we just turn off the confirmation? Is it really so dangerous to thank someone with a standardized message that confirmation is required?

swalling wrote:

I ask what I did in comment 1 because adding a preference to our already totally bloated preferences lists is undesirable.

(In reply to comment #1)

What if we just turn off the confirmation? Is it really so dangerous to thank
someone with a standardized message that confirmation is required?

I guess that should be discussed in a different bug then? Bug 47658 is what added it based on my quick skim.

I would recommend collecting some statistics to see how many people click on thanks and then hit no (accidental) versus number of people who hit yes (on purpose).

(In reply to comment #2)

I ask what I did in comment 1 because adding a preference to our already
totally bloated preferences lists is undesirable.

If there is a legitimate reason to add a preference, it shouldn't be dismissed because there are extensions/core which have added useless ones. That shouldn't stop us from holding it to higher standards though.

A proper "undo" feature would be the right way to do it. A bit tricky technically, but IMO far better than all the alternatives, so I think we should open a bug for that and close this one as WONTFIX.

I can see users undoing other users' edits when they mean to undo their own thanks.

I concur with Jackmcbam.

Another risk: if you click on undo after the editor has received the notification, it would give the false feeling it has been cancelled when it weren't.

[ Not suitable for new developer, as there are implementation issues. Removing easy keyword. ]

Or bug 47658, as mentioned in comment 3.

Mmm... so is there any plan to do something about this? It would be really nice to skip the confirmation. The reasons in the English Wikipedia thread [1] seem rather weak to me. Thanking should be more streamlined.

[1] https://en.wikipedia.org/wiki/Wikipedia:Village_pump_%28technical%29/Archive_122#Raging_thanking_dragon

(In reply to Amir E. Aharoni from comment #11)

Mmm... so is there any plan to do something about this? It would be really
nice to skip the confirmation. The reasons in the English Wikipedia thread
[1] seem rather weak to me. Thanking should be more streamlined.

[1]
https://en.wikipedia.org/wiki/Wikipedia:Village_pump_%28technical%29/
Archive_122#Raging_thanking_dragon

Those reasons were perhaps better described by Bartosz in bug 53879 (comment.5), and by MZ in bug 47658 (comment.0).

As Erik says above in this bug (comment.4), the ideal solution would be a proper "undo" feature for accidental Thanks.
The difficulty with that is, Thanks currently sends the message of appreciation *immediately*, by Echo *and by* Email (if enabled), hence developers would need to create some sort of delay system (30 or 60 seconds, were suggested elsewhere) in order for the "undo" feature to work as users want it to.
(I.e. We really can't send a 2nd email saying: "Ooops, Bob has 'unThanked' your edit, so please ignore the previous email!")
As Erik concludes, that should probably be a New Bug. Feel free to file it, and/or Patches Welcome!

(In reply to Erik Moeller from comment #4)

A proper "undo" feature would be the right way to do it. A bit tricky
technically, but IMO far better than all the alternatives, so I think we
should open a bug for that and close this one as WONTFIX.

I've filed bug 69636 ("Thanks: Implement an undo feature")

The confirmation step is now much less annoying due to jquery.confirmable. WONTFIXing because there's no way we're going to add a preference for it. General discussion about a confirmation step can be on bug 53879.

swalling wrote:

(In reply to Kunal Mehta (Legoktm) from comment #14)

The confirmation step is now much less annoying due to jquery.confirmable.
WONTFIXing because there's no way we're going to add a preference for it.
General discussion about a confirmation step can be on bug 53879.

Thanks very much Kunal. I agree that this (plus considering bug 69636 as an alternative) is the right way to go.