Skip to content

crash prompt dialog has both rounded and square corners - follow up to #1841 #17722

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

Closed
LaurenWags opened this issue Aug 27, 2021 · 5 comments · Fixed by brave/brave-core#9894
Closed

Comments

@LaurenWags
Copy link
Member

Description

Crash prompt was added with #1841

On macOS Catalina this crash prompt dialog looks odd, it has both rounded and square corners:

Screen Shot 2021-08-27 at 8 24 02 AM

Looking at the verification notes under #1841 (comment), the prompt looks ok with macOS 12.0, but Windows and Linux behave the same as Catalina having both rounded and squared corners on this prompt.

Steps to Reproduce

Follow test plan for #1841 or:

  1. Clean profile 1.29.x
  2. Open Guest window
  3. Go to brave://settings
  4. Click on Rewards at the top of the settings page
  5. Brave crashes (known issue, logged with Remove topbar and non-search settings from brave://settings for guest windows #15053)
  6. Relaunch Brave
  7. Get crash report prompt

Actual result:

Light Dark
Screen Shot 2021-08-27 at 8 41 11 AM Screen Shot 2021-08-27 at 8 24 02 AM

Expected result:

Rounded or squared corners on the prompt, not both

Reproduces how often:

Easily

Brave version (brave://version info)

Brave | 1.29.74 Chromium: 93.0.4577.58 (Official Build) (x86_64)
-- | --
Revision | c4410ece044414ea42fa4ba328d08195e818a99c-refs/branch-heads/4577@{#1076}
OS | macOS Version 10.15.7 (Build 19H1323)

Version/Channel Information:

  • Can you reproduce this issue with the current release? 1.28.x n/a, but 1.29.x yes
  • Can you reproduce this issue with the beta channel? yes
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields?
  • Does the issue resolve itself when disabling Brave Rewards?
  • Is the issue reproducible on the latest version of Chrome?

Miscellaneous Information:

cc @brave/legacy_qa @rebron @simonhong

@LaurenWags
Copy link
Member Author

This prompt also looks ok on macOS Big Sur:

Brave | 1.29.74 Chromium: 93.0.4577.58 (Official Build) (x86_64)
-- | --
Revision | c4410ece044414ea42fa4ba328d08195e818a99c-refs/branch-heads/4577@{#1076}
OS | macOS Version 11.5.2 (Build 20G95)

Screen Shot 2021-08-27 at 8 58 00 AM

@simonhong simonhong self-assigned this Aug 27, 2021
@simonhong
Copy link
Member

Hmm, this seems regression. I'll investigate.
From the original issue description history, I found captured screenshot from Windows and it was fine.
I also can repro this on my Windows now.

Screen Shot 2021-08-27 at 10 24 36 PM

@kjozwiak
Copy link
Member

Reproducible on Win 10 x64 as well. Example using the following build:

Brave | 1.29.74 Chromium: 93.0.4577.58 (Official Build) (64-bit)
-- | --
Revision | c4410ece044414ea42fa4ba328d08195e818a99c-refs/branch-heads/4577@{#1076}
OS | Windows 10 OS Version 2009 (Build 19042.1165)

image

simonhong added a commit to brave/brave-core that referenced this issue Aug 30, 2021
fix brave/brave-browser#17722

This dialog has rounded andsquared corners both.
Instead of fixing this issue for all each dialogs, we should find
the way to apply rounded corners globally.
So, just deleted code for rounded corner.
@simonhong simonhong added this to the 1.31.x - Nightly milestone Aug 30, 2021
@LaurenWags LaurenWags added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Sep 1, 2021
@LaurenWags
Copy link
Member Author

LaurenWags commented Sep 1, 2021

Verified passed on macOS x64 10.15.7 (Catalina) with

Brave | 1.31.12 Chromium: 93.0.4577.58 (Official Build) nightly (x86_64)
-- | --
Revision | c4410ece044414ea42fa4ba328d08195e818a99c-refs/branch-heads/4577@{#1076}
OS | macOS Version 10.15.7 (Build 19H1323)

Verified test plan from brave/brave-core#9894 (comment)
Confirmed no rounded corners on the prompt

Screen Shot 2021-09-01 at 2 36 14 PM


Verified PASSED on

Brave 1.31.18 Chromium: 93.0.4577.63 (Official Build) nightly (64-bit)
Revision ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS Linux

Verified test plan from brave/brave-core#9894 (comment)
Confirmed no rounded corners on the prompt

Screen Shot 2021-09-03 at 6 37 09 PM


Verified PASSED on

Brave 1.31.31 Chromium: 93.0.4577.63 (Official Build) nightly (64-bit)
Revision ff5c0da2ec0adeaed5550e6c7e98417dac77d98a-refs/branch-heads/4577@{#1135}
OS Windows 10 OS Version 2009 (Build 22000.176)

Verified test plan from brave/brave-core#9894 (comment)
Confirmed no rounded corners on the prompt

crash-dialog

@LaurenWags LaurenWags added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Sep 1, 2021
@User198263321
Copy link

User198263321 commented Nov 8, 2021

Would be nice if this could be changed with a flag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment