486420 - Enable preferences to inhibit user whitelisting

Status: RESOLVED FIXED

(Thunderbird :: Preferences, defect)

Description

[Kent James (:rkent)]

As a followup to bug 452879, we need to decide whether to enable the suppression of junk whitelisting of the user's email and domain, and whether we need UI for it.

My initial inclination is to provide checkboxes for these two preferences in the UI (though I cringe at the wording of this chain of negatives).

As to defaults, I would suggest that we enable inhibitWhiteListingIdentityUser but not inhibitWhiteListingIdentityDomain. The matching user is much more common than the matching domain, and you would generally know of any important emails that you sent yourself. The domain matching is less common, and more likely to be someone that you would want to whitelist even at the risk of the occasional spam.

Comment 1

[Kent James (:rkent)]

Attachment: Enable preference inhibitWhiteListingIdentityUser

I'm encouraging Bryan to add UI for this in bug 482617, which I will mark blocking of this bug. But the decision to enable the preference is unrelated to the UI, so let's just turn it on.

Dmose, you haven't been involved in this bug thread before, but I thought it would be fun to get you to do a review.

Comment 2

[Dan Mosedale (:dmosedale, :dmose)]

Comment on attachment 372978 [details] [diff] [review]
Enable preference inhibitWhiteListingIdentityUser

r+sr=dmose; since this is changing user-visible behavior, it needs ui-review as well, so I'm requesting that of Bryan.  Thanks for the patch!

Comment 3

[Kent James (:rkent)]

(In reply to comment #2)
> r+sr=dmose; since this is changing user-visible behavior, it needs ui-review as
> well, so I'm requesting that of Bryan.  Thanks for the patch!

While you are free to request review of anyone that you want, I don't see how you can call it "UI review" when the patch has no UI, nor affects any UI. Yes it affects program behaviour, but by that logic so does fixing a crash.

Comment 4

[Bryan Clark (DevTools PM) [:clarkbw]]

Comment on attachment 372978 [details] [diff] [review]
Enable preference inhibitWhiteListingIdentityUser

I think the term ui-review is a bit misleading.  Behaviour is the real UI, not the widgets that are drawn to the screen.  Generally the hardest "UI problems" are driven by poor behaviour and not by lack of good widgets.

Anyway, this sounds reasonable to me and lets deal with the possible widgets in bug 482617

Comment 5

[Dan Mosedale (:dmosedale, :dmose)]

Yeah, that flag should probably be named ue-review, since it's about reviewing changes to the user experience, not necessarily to the interface per se.

Comment 6

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/3642a14d6f7c