Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

browserButton should have -webkit-app-region: no-drag by default #11111

Closed
luixxiul opened this issue Sep 23, 2017 · 0 comments
Closed

browserButton should have -webkit-app-region: no-drag by default #11111

luixxiul opened this issue Sep 23, 2017 · 0 comments

Comments

@luixxiul
Copy link
Contributor

luixxiul commented Sep 23, 2017

Test plan

#11110 (comment)


Description

browserButton should be explicitly specified with -webkit-app-region: no-drag (which is the default value for buttons) to overwrite an inherited value, especially on the navigation bar.

Steps to Reproduce

  1. Open brave.com
  2. Enable NoScript
  3. Enable an extension

Actual result:

NoScript button (which is currently span element), extension button, shield button, publisher button's container are specified with no-drag individually.

Expected result:
browserButton should be speficied with -webkit-app-region: no-drag.

Reproduces how often: [What percentage of the time does it reproduce?]

Additional Information

Requires #10951 to make the shield button non-draggable

@luixxiul luixxiul self-assigned this Sep 23, 2017
@luixxiul luixxiul added misc/button polish Nice to have — usually related to front-end/visual tasks. labels Sep 23, 2017
@luixxiul luixxiul added this to the 0.21.x (Nightly Channel) milestone Sep 23, 2017
@bbondy bbondy modified the milestones: 0.21.x (Developer Channel), 0.20.x (Beta Channel) Oct 25, 2017
syuan100 pushed a commit to syuan100/browser-laptop that referenced this issue Nov 9, 2017
Closes brave#11111
Requires brave#10951 to make the shield button non-draggable

Auditors: @cezaraugusto

Test Plan:
1. Open brave.com
2. Enable NoScript
3. Enable an extension
4. Make sure NoScript button, the extension button, and publisher button is not draggable
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants