Merge pull request #368 from CodeNow/cross-browser-fixes

Cross browser fixes

Author: runnabro

client/assets/styles/scss/popover/popover-repo-actions.scss

@@ -1,6 +1,6 @@
-.repository-actions .popover {
+.popover-repo-actions {
+  min-width: 225px;
   transform-origin: 50% 0;
-  width: 225px;
 }
 
 // small.popover-list-item-details

client/assets/styles/scss/views/views-tabs.scss

@@ -37,11 +37,6 @@
     color: $gray-darker;
   }
 
-  &.active .tab-name::before {
-    background: $white;
-    content: attr(data-title);
-  }
-
   &.active {
     box-shadow: 0 0 9px rgba($black,.075);
 
@@ -52,10 +47,20 @@
     &:last-child {
       box-shadow: 9px 0 0 $white, 0 0 9px rgba($black,.075);
     }
+
+    .tab-name::before {
+      background: $white;
+      content: attr(data-title);
+    }
+
+    .icons-dirty {
+      margin-left: 9px;
+    }
   }
 
   &.dirty .icons-dirty {
-    opacity: 1;
+    opacity: .5;
+    visibility: visible;
   }
 
   &:last-child {
@@ -73,11 +78,14 @@
 
   .icons-dirty {
     display: inline-block;
+    float: left;
     font-size: 21px;
     line-height: 1.9;
     opacity: 0;
     pointer-events: none;
     position: relative;
+    user-select: none;
+    visibility: hidden;
     z-index: 1;
   }
 

client/assets/styles/scss/views/views-themes.scss

@@ -6,9 +6,8 @@
   box-shadow: 0 1px 10px rgba($black,.1) !important;
   font-family: 'Helvetica Neue', Helvetica, Arial, sans-serif;
   letter-spacing: .3px !important;
-  margin-left: -750px;
-  margin-top: -90px;
   padding: 6px 9px !important;
+  transform: translate3d(-810px,-90px,0); // fix weird ace offset
 }
 
 .ace_gutter,

client/directives/editRepoCommit/viewEditRepoCommit.jade

@@ -38,7 +38,7 @@ li.repository-group-item
     pop-over(
       actions = "popoverRepoActions.actions"
       data = "popoverRepoActions.data"
-      popover-options = "{\"left\":-74,\"top\":34}"
+      popover-options = "{\"left\":-97,\"top\":34}"
       template = "viewPopoverRepoActions"
     )
 

client/directives/editRepoCommit/viewPopoverRepoActions.jade

@@ -1,4 +1,4 @@
-.popover.menu.bottom.fade(
+.popover.menu.bottom.popover-repo-actions(
   ng-class = "{'in': data.show}"
 )
   .arrow.white

client/directives/tabs/viewTabs.jade

@@ -8,8 +8,8 @@
         'always-open': item.state.alwaysOpen\
       }"
       ng-click = "openItems.activeHistory.add(item); \
-                  openItems.saveState(); \
-                  $event.stopPropagation();"
+        openItems.saveState(); \
+        $event.stopPropagation();"
       ng-repeat = "item in openItems.models"
     )
       span.tab-name(