Changeset 38

Timestamp:

06/05/2013 03:24:02 PM (12 years ago)

Author:

nacin

Message:

Trac:

• jQuery: Use prop() instead of attr()
• Make system messages a bit less ugly

Location:

sites/trunk/wordpress.org/public_html/style/trac

Files:

• wp-trac.css (modified) (1 diff)
• wp-trac.js (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/style/trac/wp-trac.css

@@ -523 +523 @@
     box-shadow: none;
 }
-span.system-message {
-    border-radius: 3px;
-}
+span.system-message,
 div.system-message {
     border-radius: 3px;
+    box-shadow: none;
+}
+#warning.system-message,
+.warning.system-message {
+    border-color: #333;
+    color: 3333;
 }
 div.system-message .trac-close-msg {

sites/trunk/wordpress.org/public_html/style/trac/wp-trac.js

@@ -64 +64 @@
             // Clear the milestone on wontfix, duplicate, worksforme, invalid
             wpTrac.field.milestone = $('#field-milestone');
-            if ( ! wpTrac.field.milestone.attr('disabled') ) {
+            if ( ! wpTrac.field.milestone.prop('disabled') ) {
                 $('#propertyform').submit( function() {
                     var action = $('input[name=action]:checked').val();
@@ -226 +226 @@
             // Update the dropdown. Core keywords also get a title attribute with their description.
             if ( keyword in coreKeywordList ) {
-                wpTrac.field.add.find('option[value=' + keyword + ']').attr('disabled', 'disabled').text('* ' + keyword);
+                wpTrac.field.add.find('option[value=' + keyword + ']').prop('disabled', true).text('* ' + keyword);
                 title = coreKeywordList[keyword];
             }
@@ -260 +260 @@
             // Update the core keyword dropdown.
             if ( keyword in coreKeywordList )
-                wpTrac.field.add.find('option[value=' + keyword + ']').attr('disabled', '').text( keyword );
+                wpTrac.field.add.find('option[value=' + keyword + ']').prop('disabled', false).text( keyword );
             wpTrac.hiddenEl.val( wpTrac.keywords.join(' ') );
             object.remove();