From 908caa53b0c00f57f1905d2d117f9c783d323616 Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Fri, 21 Dec 2007 18:11:16 -0600
Subject: [PATCH] In response to paul's comment: "one user that just tested is
 confused by the green & red signs, and think it would be better to remove
 them (they thought "green" was = OK, confirm creation and "red" = oops, I
 made a mistake, it's a duplicate)"

I've taken out the check and the X and replaced them with "edit" and "save" icons from the toolbars. Less confusing?

Signed-off-by: Chris Cormack <crc@liblime.com>
Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css | 12 ++++++++++++
 .../prog/en/modules/cataloguing/addbiblio.tmpl       |  4 ++--
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
index bd12796b3b..f023494275 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
+++ b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
@@ -1054,12 +1054,24 @@ div.first fieldset {
  
 .dialog input.approve {
  	background : #FFF url(../../img/approve.gif) no-repeat 4px center;
+	padding : .4em .4em .4em 25px;	
  }
 
 .dialog input.deny {
  	background : #FFF url(../../img/deny.gif) no-repeat 4px center;
+	padding : .4em .4em .4em 25px;
  }
+
+.dialog input.save {
+	background: #fff url(../../img/toolbar-save.gif) no-repeat 4px center;
+	padding: .4em .4em .4em 34px;
+}
  
+.dialog input.edit {
+	background: #fff url(../../img/toolbar-edit.gif) no-repeat 4px center;
+	padding: .4em .4em .4em 34px;
+}
+
 .dialog input:hover {
  	background-color : #ffc;
  }
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tmpl
index 1b753423ce..3be7e525f9 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tmpl
@@ -447,10 +447,10 @@ function unHideSubfield(index,labelindex) { // FIXME :: is it used ?
                         <p>Is this a duplicate of <a href="/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=<!-- TMPL_VAR name="duplicatebiblionumber" -->" onclick="openWindow('../MARCdetail.pl?biblionumber=<!-- TMPL_VAR name="duplicatebiblionumber" -->&amp;popup=1', 'Duplicate biblio'; return false;)"><!-- TMPL_VAR name="duplicatetitle" --></a>?</p>
                         <form action="/cgi-bin/koha/cataloguing/additem.pl" method="get">
                             <input type="hidden" name="biblionumber" value="<!-- TMPL_VAR name="duplicatebiblionumber" -->" />
-                            <input type="submit" class="approve" value="Yes: Edit existing items" />
+                            <input type="submit" class="edit" value="Yes: Edit existing items" />
                         </form>
                         <form action="/cgi-bin/koha/cataloguing/addbiblio.pl" method="get">
-                            <input type="submit" class="deny" onclick="confirmnotdup(); return false;" value="No: Save as New Record" />
+                            <input type="submit" class="save" onclick="confirmnotdup(); return false;" value="No: Save as New Record" />
                         </form>
                     </div>
         <!-- /TMPL_IF -->
-- 
2.20.1