Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Local files should not be shown with "insecure" icon in the UI #35316

Closed
ZelnickB opened this issue Jan 15, 2024 · 10 comments · Fixed by brave/brave-core#21722
Closed

Local files should not be shown with "insecure" icon in the UI #35316

ZelnickB opened this issue Jan 15, 2024 · 10 comments · Fixed by brave/brave-core#21722
Assignees
Labels

Comments

@ZelnickB
Copy link

When you open a local file (e.g., a PDF) with Brave (I have tested it with Version 1.61.116 Chromium: 120.0.6099.217 (Official Build) (arm64) on MacOS Apple Silicon), the "insecure" icon used for HTTP sites (insecure origins) is displayed in the address bar:
Insecure (exclamation point) icon shown next to "File" label in address bar.

Local files are not "insecure" like an HTTP site, so perhaps Brave should use an icon like that used in Google Chrome (with an "info" or similar icon appearing next to the "File" label).
"Info" icon shown next to 'File' label in address bar when using Google Chrome.

@ZelnickB
Copy link
Author

Somewhat related to #33755.

@diracdeltas
Copy link
Member

cc @rebron @petemill @bradleyrichter does anyone know if this was an intentional design change on our part from Chrome?

@rebron rebron added priority/P2 A bad problem. We might uplift this to the next planned release. QA/Yes release-notes/include labels Jan 22, 2024
@rebron rebron added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P2 A bad problem. We might uplift this to the next planned release. labels Jan 22, 2024
@simonhong simonhong self-assigned this Jan 23, 2024
@simonhong
Copy link
Member

When I load http site different icon is used (rounded triangle)
image

and circle icon for file scheme.
image

Is there another way to see circle icon for http site?

@diracdeltas
Copy link
Member

@simonhong that's a good point. I think the issue is the exclamation point (!) is associated with insecure sites even though it's not exactly the same UI as insecure. if it was the info icon (i) instead it'd be fine.

@bradleyrichter
Copy link

@simonhong swapping over to the info icon would work here

@simonhong
Copy link
Member

I found that we have another info icon with i from our icon set (https://www.figma.com/file/g8Z0q6TMPYDq6zXh9Y7LWD/%F0%9F%8C%8E-Universal?type=design&node-id=1419-13501&mode=design&t=Ae5NaXQUaIR2C1LK-0)
Screenshot 2024-01-23 at 11 31 04 AM
Maybe it's good to use here?

@diracdeltas
Copy link
Member

lgtm

@aguscruiz
Copy link

@simonhong let's used the filled variant there, since the ⚠️ icon is also filled. 👌

@kjozwiak
Copy link
Member

The above requires 1.62.155 or higher for 1.62.x verification 👍

@LaurenWags
Copy link
Member

LaurenWags commented Jan 30, 2024

Verified with

Brave | 1.62.155 Chromium: 121.0.6167.85 (Official Build) (x86_64)
-- | --
Revision | 91c989228daf153ef41bd965f268fabc7b9dbbf0
OS | macOS Version 13.6.4 (Build 22G513)

Reproduced the issue using 1.62.153 Chromium: 121.0.6167.85.

Using 1.62.155 Chromium: 121.0.6167.85 confirmed that the location bar/omnibox was displaying i rather than ! when opening local files.
Confirmed for both dark and light themes.

1.62.153 1.62.155 (dark) 1.62.155 (light)
1 2 3

@LaurenWags LaurenWags added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Jan 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants