Skip to content

improve/fix default "focus" element for ToS notification under rewards panel #51133

@kjozwiak

Description

@kjozwiak

Description

We should improve the default "focus" element via the ToS notification that appears within the rewards panel. Currently we're selecting reset as the initial focused element and using a large/black border that makes it really stick out like a sore thumb. Think we can probably do better.

Steps to reproduce

Prerequisite: ensure that there's a ToS notification active/visible within the rewards panel

  1. ensured that you have a rewards verified account (custodian/wallet etc..)
  2. as per the above prerequisite, ensure there's a ToS notification with the rewards panel
  3. click/tap on the rewards panel and you'll notice that reset as the first element that's focused with a large border

Actual result

Example Example
Image Image

Expected result

We should probably make the I agree button the default focused element and fix the large/black border.

Auto-focus should just be on the dialog / modal itself. cc: @zenparsing

Reproduces how often

Easily reproduced

Desktop Brave version (brave://version info)

Brave | 1.87.26 Chromium: 143.0.7499.40 (Official Build) nightly (64-bit)
-- | --
Revision | aac661753357a96cb3ef6be06df8c091b8eaa62f
OS | Windows 11 Version 25H2 (Build 26200.7171)

Android device

  • Brand/model: Samsung S25 Note Ultra
  • Android version: Android 16

Channel information

  • release (stable)
  • beta
  • nightly

Reproducibility

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

Miscellaneous information

No response

Metadata

Metadata

Assignees

Type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions