Skip to content

Commit

Permalink
Dialog demo: Use the ui-state-highlight class instead of a hard-coded…
Browse files Browse the repository at this point in the history
… color. Fixes #4808 - Modal dialog form validation demo - error message uses hard-coded color.
  • Loading branch information
scottgonzalez committed Aug 29, 2009
1 parent 8d2caca commit da0aece
Showing 1 changed file with 11 additions and 6 deletions.
17 changes: 11 additions & 6 deletions demos/dialog/modal-form.html
Expand Up @@ -11,7 +11,6 @@
<script type="text/javascript" src="../../ui/ui.stackfix.js"></script>
<script type="text/javascript" src="../../ui/ui.dialog.js"></script>
<script type="text/javascript" src="../../ui/effects.core.js"></script>
<script type="text/javascript" src="../../ui/effects.highlight.js"></script>
<link type="text/css" href="../demos.css" rel="stylesheet" />
<style type="text/css">
body { font-size: 62.5%; }
Expand All @@ -23,8 +22,8 @@
div#users-contain table { margin: 1em 0; border-collapse: collapse; width: 100%; }
div#users-contain table td, div#users-contain table th { border: 1px solid #eee; padding: .6em 10px; text-align: left; }
.ui-button { outline: 0; margin: 0; padding: .4em 1em .5em; text-decoration: none; cursor: pointer; position: relative; text-align: center; }
.ui-dialog .ui-state-highlight, .ui-dialog .ui-state-error { padding: .3em; }

.ui-dialog .ui-state-error { padding: .3em; }
.validateTips { border: 1px solid transparent; padding: 0.3em; }

</style>
<script type="text/javascript">
Expand All @@ -36,10 +35,15 @@
email = $("#email"),
password = $("#password"),
allFields = $([]).add(name).add(email).add(password),
tips = $("#validateTips");
tips = $(".validateTips");

function updateTips(t) {
tips.text(t).effect("highlight",{},1500);
tips
.text(t)
.addClass('ui-state-highlight');
setTimeout(function() {
tips.removeClass('ui-state-highlight', 1500);
}, 500);
}

function checkLength(o,n,min,max) {
Expand Down Expand Up @@ -70,6 +74,7 @@
stackfix: true,
autoOpen: false,
height: 300,
width: 350,
modal: true,
buttons: {
'Create an account': function() {
Expand Down Expand Up @@ -137,7 +142,7 @@
<div class="demo">

<div id="dialog-form" title="Create new user">
<p id="validateTips">All form fields are required.</p>
<p class="validateTips">All form fields are required.</p>

<form>
<fieldset>
Expand Down

0 comments on commit da0aece

Please sign in to comment.