Permalink
Browse files

[Fixes #12849447] Clarify when item unavailable for SCO renewal

UI ambiguity was contributing to a situation where it was easy for
a patron to think they were renewing an item when in fact they
were choosing to indicate they had returned it. A working button
in a table column labeled "Renew" should only be able to renew
items, rather than quietly presenting the very similar option to
"Return Item". This patch disables one-click returns and makes
the item's unavailability more visually apparent.
  • Loading branch information...
1 parent 342741c commit 719a562796977091d54e6278d3d1c8239ca8c529 @ctfliblime ctfliblime committed Jun 1, 2011
@@ -78,6 +78,14 @@ input[type=submit]:active, input[type=button], input[type=reset] {
border: 1px inset #666666;
float : none;
}
+
+input[type=button].unavailable {
+ background : #aaa;
+ color : #111;
+ font-weight : normal;
+ font-style : italic;
+}
+
.dialog {
border: 1px solid #bcbcbc;
padding : .5em;
@@ -209,7 +209,7 @@ Sorry, This Self-Checkout Station has lost authentication. Please contact the a
<input type="hidden" name="barcode" value="<!-- TMPL_VAR NAME="barcode" -->" />
<input type="hidden" name="confirmed" value="" />
<!-- TMPL_IF NAME="norenew" -->
- <input type="button" value="Return Item" name="confirm" class="return" onclick="this.form.op.value='returnbook';this.form.submit();" />
+ <input type="button" value="Unavailable" disabled="1" name="confirm" class="unavailable" />
<!-- TMPL_ELSE -->
<input type="button" value="Renew Item" <!-- TMPL_UNLESS NAME="renew" --> name="confirm"<!-- /TMPL_UNLESS --> class="renew" onclick="this.form.confirmed.value='1';this.form.submit();" />
<!-- /TMPL_IF -->

0 comments on commit 719a562

Please sign in to comment.