Prevent Tab Close Button from allowing tab-tearoff

Auditors: @luixxiul Close #9511
brave · Jun 24, 2017 · 1500ecd · 1500ecd
1 parent e3eab8a
commit 1500ecd
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/app/renderer/components/tabs/content/closeTabIcon.js b/app/renderer/components/tabs/content/closeTabIcon.js
@@ -31,6 +31,7 @@ class CloseTabIcon extends React.Component {
   constructor (props) {
     super(props)
     this.onClick = this.onClick.bind(this)
+    this.onDragStart = this.onDragStart.bind(this)
   }
 
   get frame () {
@@ -48,6 +49,10 @@ class CloseTabIcon extends React.Component {
     }
   }
 
+  onDragStart (event) {
+    event.preventDefault()
+  }
+
   mergeProps (state, ownProps) {
     const currentWindow = state.get('currentWindow')
     const isPinnedTab = frameStateUtil.isPinned(currentWindow, ownProps.frameKey)
@@ -76,6 +81,8 @@ class CloseTabIcon extends React.Component {
       className={css(this.props.showCloseIcon && styles.closeTab)}
       l10nId='closeTabButton'
       onClick={this.onClick}
+      onDragStart={this.onDragStart}
+      draggable='true'
     />
   }
 }

diff --git a/app/renderer/components/tabs/content/tabIcon.js b/app/renderer/components/tabs/content/tabIcon.js
@@ -33,6 +33,8 @@ class TabIcon extends ImmutableComponent {
     return <div
       className={this.props.className}
       data-test-favicon={this.props['data-test-favicon']}
+      onDragStart={this.props.onDragStart}
+      draggable={this.props.draggable}
       onClick={this.props.onClick}>
       {
         this.props.symbol