replaced image DialogCloser and removed legacy code

app/assets/javascripts/modal.js

@@ -242,7 +242,7 @@ var ModalHelper = (function() {
 
     if (this._firstLoad) {
       //add closer
-      modalDiv.parent().prepend('<div id="ui-dialog-closer" />').click(jQuery.proxy(this.close, this));
+      modalDiv.parent().prepend('<div id="ui-dialog-closer" class="icon icon-delete2" />').click(jQuery.proxy(this.close, this));
       jQuery('.ui-dialog-titlebar').hide();
       jQuery('.ui-dialog-buttonpane').hide();
 

app/assets/stylesheets/_misc_legacy.sass

@@ -334,13 +334,6 @@ div.issue hr
   margin-top: -6px
   margin-bottom: 0
 
-.attachments h4
-  margin-bottom: 6px
-  background: image-url("files-showhide.png") no-repeat right bottom
-  cursor: pointer
-  &.closed
-    background-position: right 5px
-
 table.files
   display: table
 
@@ -398,22 +391,6 @@ ul.projects
   top: 7px
   right: 14px
 
-/* file table hovers */
-
-a.has-thumb
-  img
-    position: absolute
-    display: none
-    border: 1px solid #a6c6cf
-    padding: 4px
-    background: #fff
-    border-radius: 3px
-  &.active
-    left: -10px
-    background: image-url("thumb-arrow-right.png") no-repeat left center
-    padding-left: 10px
-    margin-right: -10px
-
 td.issue div.issue-wrap-outer
   position: relative
 
@@ -627,8 +604,6 @@ div.indent
       padding-top: 0
 
 #content
-  #history
-    background: image-url("dotted-separator.gif") repeat-x scroll 0 bottom transparent
   .wiki-content
     p, li
       width: 700px
@@ -639,14 +614,15 @@ input::-webkit-input-placeholder, :-moz-placeholder
   color: #000000
 
 #ui-dialog-closer
-  background-image: image-url("modal_close.png")
   cursor: pointer
-  height: 33px
   position: absolute
-  right: -15px
-  top: -15px
-  width: 33px
+  right: -21px
+  top: -17px
   z-index: 5000
+  font-size: 2rem
+
+  &.icon:before
+    color: black
 
 h4.comment
   margin-bottom: 8px

app/assets/stylesheets/layout/_work_package.sass

@@ -47,6 +47,16 @@
   .flash ~ div[ui-view]
     height: calc(100% - #{($content-flash-height + $flash-margins-padding)})
 
+  // If there are two flash messages shown:
+  // Subtract the height of the two flash messages
+  .flash:not(.ng-hide) ~ .flash:not(.ng-hide) ~ div[ui-view]
+    height: calc(100% - #{2 * ($content-flash-height + $flash-margins-padding)})
+
+  #history
+    border-bottom-color: rgb(221, 221, 221)
+    border-bottom-style: solid
+    border-bottom-width: 1px
+
 // HACK: workaround to ensure correct height applied to child elements
 #work-packages-index
   height: 100%