From 4c861ff00786abab1fac744fc345e9a1b0e65081 Mon Sep 17 00:00:00 2001
From: Juan Tejada <juans.tejada@gmail.com>
Date: Thu, 16 Jun 2016 15:37:21 -0700
Subject: [PATCH] fix(quoted-text): Add option to remove quoted-text, fix
 styles

- #2323
---
 .../composer/lib/composer-view.jsx            | 16 +++++++++++
 .../composer/stylesheets/composer.less        | 27 ++++++++++++++++---
 2 files changed, 40 insertions(+), 3 deletions(-)

diff --git a/internal_packages/composer/lib/composer-view.jsx b/internal_packages/composer/lib/composer-view.jsx
index c7b84e787..1e4ba1eb2 100644
--- a/internal_packages/composer/lib/composer-view.jsx
+++ b/internal_packages/composer/lib/composer-view.jsx
@@ -254,6 +254,13 @@ export default class ComposerView extends React.Component {
       return (
         <a className="quoted-text-control" onClick={this._onToggleQuotedText}>
           <span className="dots">&bull;&bull;&bull;</span>
+          <span className="remove-quoted-text" onClick={this._onRemoveQuotedText}>
+            <RetinaImg
+              title="Remove quoted text"
+              name="image-cancel-button.png"
+              mode={RetinaImg.Mode.ContentPreserve}
+            />
+          </span>
         </a>
       );
     }
@@ -264,6 +271,15 @@ export default class ComposerView extends React.Component {
     this.setState({showQuotedText: !this.state.showQuotedText});
   }
 
+  _onRemoveQuotedText = (event) => {
+    event.stopPropagation()
+    const {session, draft} = this.props
+    session.changes.add({
+      body: QuotedHTMLTransformer.removeQuotedHTML(draft.body),
+    })
+    this.setState({showQuotedText: false})
+  }
+
   _renderFooterRegions() {
     return (
       <div className="composer-footer-region">
diff --git a/internal_packages/composer/stylesheets/composer.less b/internal_packages/composer/stylesheets/composer.less
index 999035ca4..799956027 100644
--- a/internal_packages/composer/stylesheets/composer.less
+++ b/internal_packages/composer/stylesheets/composer.less
@@ -295,12 +295,33 @@ body.platform-win32 {
     margin: 0 8px;
 
     .quoted-text-control {
-      margin: @spacing-standard;
+      position: relative;
       // The quoted-text-control has no top margin since the
       // div[contentedtiable] has 10px of bottom padding. It's better to
       // put the padding on the contenteditable so the bottom looks nice
-      // in popout windoes when there's no quoted text control.
-      margin-top: 0;
+      // in popout windows when there's no quoted text control.
+      margin: 0 @spacing-standard @spacing-standard 0;
+
+      .remove-quoted-text {
+        display: none;
+        cursor: pointer;
+        position: absolute;
+        z-index: 2;
+        right: -6px;
+        top: -6px;
+        border-radius: 0 0 0 3px;
+        &:active {
+          background: none;
+          -webkit-filter: brightness(95%);
+        }
+        img {
+          height: 24px;
+        }
+      }
+
+      &:hover .remove-quoted-text {
+        display: block;
+      }
     }
 
     div[contenteditable] {