Skip to content

Click on Brave News (RSS) button should also close dialog #37216

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
rebron opened this issue Mar 29, 2024 · 4 comments · Fixed by brave/brave-core#24226
Closed

Click on Brave News (RSS) button should also close dialog #37216

rebron opened this issue Mar 29, 2024 · 4 comments · Fixed by brave/brave-core#24226

Comments

@rebron
Copy link
Collaborator

rebron commented Mar 29, 2024

Description

When clicking the Brave News RSS button in the url bar to open, clicking the button again does not close the dialog. Only clicking away closes the dialog. All other buttons e.g. Shields, Rewards, share and notification icons, clicking on the button is open and close of the dialog.

Steps to Reproduce

  1. Go to a site that displays the Brave News icon e.g. nytimes.com
  2. Click on the Brave News button to open
  3. Notice that clicking on the button to close doesn't close it. Only clicking away closes the dialog.

Actual result:

Screenshot 2024-03-28 at 7 58 19 PM

Expected result:

See description

Reproduces how often:

Easily.

Brave version (brave://version info)

Version/Channel Information:

  • Can you reproduce this issue with the current release? 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? n/a
  • Does the issue resolve itself when disabling Brave Rewards? n/a
  • Is the issue reproducible on the latest version of Chrome? n/a

Miscellaneous Information:

@rebron rebron added OS/Desktop priority/P3 The next thing for us to work on. It'll ride the trains. feature/brave-news formerly brave-today release-notes/include QA/Yes labels Mar 29, 2024
@rebron rebron moved this to On Deck in Front End May 28, 2024
@rebron rebron added priority/P4 Planned work. We expect to get to it "soon". good first issue and removed priority/P3 The next thing for us to work on. It'll ride the trains. labels Jun 14, 2024
@rebron
Copy link
Collaborator Author

rebron commented Jun 14, 2024

cc: @jagadeshjai Is this one something you can help us out with?

@jagadeshjai
Copy link

cc: @jagadeshjai Is this one something you can help us out with?

Yes @rebron, I'll look into this.

@Roopasdev
Copy link

Roopasdev commented Jun 21, 2024

Hi, I am actually a beginner trying to contribute but I am interested to work on it should I start of with this issue?

@rebron rebron added this to the 1.69.x - Nightly milestone Jul 14, 2024
@rebron rebron moved this from On Deck to Completed in Front End Jul 14, 2024
@stephendonner
Copy link

stephendonner commented Jul 15, 2024

Verified PASSED using

Brave | 1.69.106 Chromium: 127.0.6533.43 (Official Build) nightly (x86_64)
-- | --
Revision | 2778b12a30fa8f4d7ccd6caad101a7aabcfc6981
OS | macOS Version 11.7.10 (Build 20G1427)

Steps:

  1. installed 1.69.106
  2. launched Brave
  3. loaded newsweek.com
  4. opened a new-tab page
  5. scrolled down
  6. clicked on Turn on Brave News
  7. returned to the newsweek.com tab
  8. clicked on Manage Subscriptions
  9. clicked Follow
  10. clicked again on the Manage Subscriptions button

Confirmed clicking on the Manage Subscriptions button toggles it open/closed, as it should

news-button

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants