Issue #1908624 by jessebeach, tkoleary: Update edit modules implementation of...

Issue #1908624 by jessebeach, tkoleary: Update edit modules implementation of hook_toolbar(); refresh the design to match current mockups.

core/modules/edit/css/edit.css

@@ -81,8 +81,8 @@
 .active .icon-edit:before {
   background-image: url("../images/icon-edit-active.png");
 }
-.toolbar .tray.edit.active {
-  z-index: 340;
+.js .toolbar .bar .edit-toolbar-tab.tab {
+  float: right;
 }
 .toolbar .icon-edit.edit-nothing-editable-hidden {
   display: none;

core/modules/edit/edit.module

@@ -60,6 +60,9 @@ function edit_toolbar() {
         ),
       ),
     ),
+    '#wrapper_attributes' => array(
+      'class' => array('edit-toolbar-tab'),
+    ),
     '#attached' => array(
       'library' => array(
         array('edit', 'edit'),

core/modules/edit/js/views/menu-view.js

@@ -20,17 +20,9 @@ Drupal.edit.views.MenuView = Backbone.View.extend({
   initialize: function() {
     _.bindAll(this, 'stateChange');
     this.model.on('change:isViewing', this.stateChange);
-    // @todo
-    // Re-implement hook_toolbar and the corresponding JavaScript behaviors
-    // once https://drupal.org/node/1847198 is resolved. The toolbar tray is
-    // necessary when the page request is processed because its render element
-    // has an #attached property with the Edit module library code assigned to
-    // it. Currently a toolbar tab is not passed as a renderable array, so
-    // #attached properties are not processed. The toolbar tray DOM element is
-    // unnecessary right now, so it is removed.
-    this.$el.find('#toolbar-tray-edit').remove();
-    // Respond to clicks on other toolbar tabs. This temporary pending
-    // improvements to the toolbar module.
+    // Respond to clicks on other toolbar tabs. 
+    // @todo This temporary pending improvements to the toolbar module.
+    // @see https://drupal.org/node/1860434
     $('#toolbar-administration').on('click.edit', '.bar a:not(#toolbar-tab-edit)', _.bind(function (event) {
       this.model.set('isViewing', true);
     }, this));