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

[CR 126] Update brave notification text is not visible and update brave icon is changed #38645

Closed
1 of 6 tasks
GeetaSarvadnya opened this issue May 28, 2024 · 8 comments · Fixed by brave/brave-core#23931
Closed
1 of 6 tasks

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented May 28, 2024

Description

Found while testing #37788
Update brave notification text is not visible and update brave icon is changed

Steps to reproduce

  1. Clean profile 1.68.50 CR 126
  2. Open brave://settings/help
  3. Wait for an update don't click on Relaunch
  4. Update brave notification is shown
  5. Update brave notification text is not visible and update brave icon is changed

Actual result

Update brave notification text is not visible and update brave icon is changed

image

Expected result

Install 1.68.45 - CR 125, the build before CR126 nightly build, the update brave notification is shown as expected
image (5)

Reproduces how often

Easily reproduced

Brave version (brave://version info)

Brave 1.68.50 Chromium: 126.0.6478.17 (Official Build) nightly (64-bit)
Revision 72b186fbe4e288f53b0ee102f6ec66133b41a992
OS Windows 10 Version 22H2 (Build 19045.4412)

Channel information

  • release (stable)
  • beta
  • nightly

Reproducibility

  • with Brave Shields disabled
  • with Brave Rewards disabled
  • in the latest version of Chrome

Miscellaneous information

cc: @brave/qa-team @emerick @rebron @mherrmann

@rebron rebron added this to General May 28, 2024
@rebron rebron moved this to On Deck in General May 28, 2024
@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label May 28, 2024
@kjozwiak
Copy link
Member

Just adding some more context/data. Seeing the same issue in Win 11 x64 using the following build(s):

Brave | 1.68.58 Chromium: 126.0.6478.17 (Official Build) nightly (64-bit)
-- | --
Revision | ec786b623be34c911487c24144c0650bdd057fc1
OS | Windows 11 Version 23H2 (Build 22631.3672)

image

@emerick
Copy link
Contributor

emerick commented May 30, 2024

@simonhong Will probably leave this one in your hands if that's OK.

@simonhong
Copy link
Member

working on this.

@simonhong simonhong moved this from On Deck to In progress in General May 31, 2024
@rebron rebron changed the title CR 126 - Update brave notification text is not visible and update brave icon is changed [CR 126] Update brave notification text is not visible and update brave icon is changed May 31, 2024
simonhong added a commit to brave/brave-core that referenced this issue May 31, 2024
@github-project-automation github-project-automation bot moved this from In progress to Completed in General Jun 1, 2024
@brave-builds brave-builds added this to the 1.68.x - Nightly milestone Jun 1, 2024
@bsclifton
Copy link
Member

bsclifton commented Jun 4, 2024

Ran into this today (build before fix) - nice work @simonhong 😄 And thanks for logging @GeetaSarvadnya

@kjozwiak
Copy link
Member

kjozwiak commented Jun 5, 2024

The above requires 1.67.106 or higher for 1.67.x verification 👍 Just a quick note, created #38810 in case we want to improve the highlight/selection UX/UI. However, Chrome looks the same so we might just want to leave it. @simonhong provided the following image re: Chrome having the same UI/UX experience:

image

@LaurenWags LaurenWags added OS/macOS OS/Windows QA/In-Progress Indicates that QA is currently in progress for that particular issue and removed QA/Test-All-Platforms labels Jun 6, 2024
@LaurenWags
Copy link
Member

LaurenWags commented Jun 6, 2024

Verified with

Brave	1.67.107 Chromium: 126.0.6478.26 (Official Build) beta (arm64) 
Revision	2831369c7540d93e98f38d9b463f6762430a7982
OS	macOS Version 14.5 (Build 23F79)

Installed above version, confirmed that the update to 1.67.108 naturally displayed on the hamburger menu:

Image 6-10-24 at 10 09 AM

@LaurenWags
Copy link
Member

LaurenWags commented Jun 7, 2024

Verified with

Brave | 1.67.107 Chromium: 126.0.6478.26 (Official Build) beta (x86_64)
-- | --
Revision | 2831369c7540d93e98f38d9b463f6762430a7982
OS | macOS Version 13.6.7 (Build 22G720)

Reproduced the issue using 1.67.105:

Dark Light
Screenshot 2024-06-07 at 8 56 09 AM Screenshot 2024-06-07 at 9 02 32 AM

Using 1.67.107:

  • Launched via terminal with --args --enable-logging=stderr --simulate-critical-update
  • Confirmed no red/orange coloring on menu item (hamburger menu with "Update" text is red as expected)
  • Confirmed expected icon on hamburger menu item for "Relaunch to Update Brave"
Dark Light
Screenshot 2024-06-07 at 8 54 37 AM Screenshot 2024-06-07 at 9 04 45 AM

Encountered:

@LaurenWags LaurenWags added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Jun 7, 2024
@MadhaviSeelam
Copy link

Verification PASSED using

Brave | 1.67.107 Chromium: 126.0.6478.26 (Official Build) beta (64-bit)
-- | --
Revision | 2831369c7540d93e98f38d9b463f6762430a7982
OS | Windows 11 Version 23H2 (Build 22631.3672)

Reproduced the issue using 1.67.105:

Light Dark
image image

Using 1.67.107:

  • Launched via terminal with --enable-logging=stderr --simulate-critical-upgrade
  • Confirmed no red/orange coloring on menu item
  • Confirmed expected icon on hamburger menu item for "Relaunch to Update Brave"
Light Dark
image image

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