brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.74k stars 2.31k forks source link

Location bar button is not squircle when permission bubble is shown #39561

Closed simonhong closed 3 months ago

simonhong commented 3 months ago

Description

Get notifications? button doesn't have our radius and bg color

image

Left(cr127) vs. right(cr126) image

Steps to reproduce

  1. Load https://www.bennish.net/web-notifications.html
  2. Click Authorize

Actual result

See the above screen shot

Expected result

It should have our radius and bg color

Screenshot 2024-07-04 at 11 00 31 AM

Reproduces how often

Easily reproduced

Brave version (brave://version info)

1.69.76 Chromium: 127.0.6533.26 (Official Build) nightly (x86_64)

Channel information

Reproducibility

Miscellaneous information

Note: stable/beta 's bg color seems fine. nightly has different bg color. cc @rebron @brave/qa-team

simonhong commented 3 months ago

checking now.

kjozwiak commented 3 months ago

The above requires 1.68.118 or higher for 1.68.x verification 👍

LaurenWags commented 3 months ago

Verified with

Brave | 1.68.118 Chromium: 127.0.6533.26 (Official Build) beta (x86_64)
-- | --
Revision | 2863b4f7f2c566e828c816caafbc2a2ce5fb527f
OS | macOS Version 13.6.7 (Build 22G720)

Encountered https://github.com/brave/brave-browser/issues/39719 while testing.

Reproduced the issue using 1.68.115.

Using 1.68.118, confirmed the Get notifications? button and subsequent "Allowed" or "Blocked" button is now our "squircle" shape. Using 1.68.118, confirmed the blueish tint is removed.

Light mode

1.68.115 1.68.118
Screenshot 2024-07-12 at 9 42 44 AM Screenshot 2024-07-12 at 9 48 08 AM
Screenshot 2024-07-12 at 9 42 52 AM Screenshot 2024-07-12 at 9 48 18 AM
Screenshot 2024-07-12 at 9 43 05 AM Screenshot 2024-07-12 at 9 48 28 AM
Screenshot 2024-07-12 at 9 43 21 AM Screenshot 2024-07-12 at 9 48 42 AM
Screenshot 2024-07-12 at 9 43 30 AM Screenshot 2024-07-12 at 9 48 56 AM

Dark mode

1.68.115 1.68.118
Screenshot 2024-07-12 at 9 39 46 AM Screenshot 2024-07-12 at 9 46 05 AM
Screenshot 2024-07-12 at 9 40 00 AM Screenshot 2024-07-12 at 9 46 12 AM
Screenshot 2024-07-12 at 9 40 16 AM Screenshot 2024-07-12 at 9 46 21 AM
Screenshot 2024-07-12 at 9 40 52 AM Screenshot 2024-07-12 at 9 46 39 AM
Screenshot 2024-07-12 at 9 41 02 AM Screenshot 2024-07-12 at 9 46 49 AM