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

0.12.2dev titlebar fixes (round 1) #4243

Merged
merged 1 commit into from
Sep 23, 2016
Merged

0.12.2dev titlebar fixes (round 1) #4243

merged 1 commit into from
Sep 23, 2016

Conversation

bsclifton
Copy link
Member

Don't set allowDragging class for Windows on:

  • bookmarks toolbar
  • tab pages control
  • tabs toolbar

Cleaner fix than marking class as no-drag !important w/ CSS

Fixes #4171
Should fix #4232

Auditors: @bbondy (important that you try it- I couldn't reproduce 4232)

- bookmarks toolbar
- tab pages control
- tabs toolbar

Cleaner fix than marking class as `no-drag !important` w/ CSS

Fixes #4171
Should fix #4232
@bbondy
Copy link
Member

bbondy commented Sep 23, 2016

++ but it would be great to re-add these areas as draggable in the future. Pls post an issue to investigate how to suppress the system menu for later investigation

@bbondy bbondy merged commit f51e1a3 into brave:master Sep 23, 2016
@bsclifton
Copy link
Member Author

Issue created #4244

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