1194874 - Maybe switch "run by" text to "owned by" for EV certificates in the Control Center

Status: RESOLVED FIXED

(Firefox :: Security, defect, P3)

Description

[Brian Grinstead [:bgrins]]

In Bug 1191044 we removed the 'Firefox verified that' portion of the text about EV certificates.  There was a suggestion that 'run by' should switch to 'owned by' also.  Moving that discussion into this bug.

(April King from comment #10)
> That text works for me, although I would note that it's more accurate to say
> "owned by" than it is to say "run by", since most sites are not run by the
> person who owns them.

Comment 1

[Tanvi Vyas[:tanvi]]

"operated by" might work too.

Comment 2

[Anne (:annevk)]

Note that if you click "More Information" it says "Owner". However, I think we should get rid of this sliding dialog altogether. It's not worth the complexity given that "More Information" exists. I filed bug 1194938 on that.

Comment 3

[Brian Grinstead [:bgrins]]

(In reply to Anne (:annevk) from comment #2)
> Note that if you click "More Information" it says "Owner". However, I think
> we should get rid of this sliding dialog altogether. It's not worth the
> complexity given that "More Information" exists. I filed bug 1194938 on that.

Note that the sliding dialog also contains mixed content blocking information and control to disable / enable MCB

Comment 4

[Anne (:annevk)]

Indeed, you're correct, I think my main argument that "run by" and "verified by" are not helping the user still stand though.

I also do still think that in the majority of cases that arrow is a distraction and clicking it doesn't result in a user benefit. So perhaps we should reconsider some aspects of it.

Comment 5

[:Gijs (he/him)]

Based on:

(In reply to agrigas from bug 1194938 comment #17)
> We're going to go forward with what we have and will re-address adding in
> additional security info down the road.

is this also wontfix, or is there still something to be done here? If so, can we make this a simple mentored bug to switch to the alternative labels?

Comment 6

[Ash Grigas]

Will update to owned by

Comment 7

[Tanvi Vyas[:tanvi]]

Johann, do you want to pick this string change up?

Comment 8

[Johann Hofmann [:johannh]]

Hey Tanvi, thanks for asking! I could do it, but if it's really just a string change I would rather let someone new from the community take it :)

Comment 9

[karl]

Attachment: Change 'run by' to 'owned by' in the Control Center

Comment 10

[Johann Hofmann [:johannh]]

Comment on attachment 8756957 [details] [diff] [review]
Change 'run by' to 'owned by' in the Control Center

Hey there, thanks for contributing! This is a really tiny change that probably doesn't require tests. So we should be good to go. However, we need someone to review it.

Gijs, do you have a minute?

Comment 11

[:Gijs (he/him)]

Comment on attachment 8756957 [details] [diff] [review]
Change 'run by' to 'owned by' in the Control Center

Review of attachment 8756957 [details] [diff] [review]:
-----------------------------------------------------------------

This is close, but we need to rev the string ID (so make it identity.connectionVerified2) both here and in the place the string is used (here: https://dxr.mozilla.org/mozilla-central/rev/8d0aadfe7da782d415363880008b4ca027686137/browser/locales/en-US/chrome/browser/browser.dtd#685 ) , so that localizers to other languages will also update their strings.

Comment 12

[karl]

Attachment: Updates string ID to reflect string change

Comment 13

[:Gijs (he/him)]

Comment on attachment 8757021 [details] [diff] [review]
Updates string ID to reflect string change

Review of attachment 8757021 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me, thanks!

Comment 14

[:Gijs (he/him)]

Setting checkin-needed so the sheriffs will land this for us. :-)

Comment 15

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/f2125a888d23

Comment 16

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/f2125a888d23