Changeset 333 for Dev/branches/rest-dojo-ui/client/rft

Timestamp:

05/23/12 18:05:17 (13 years ago)

Author:

tjcschipper

Message:

• Added rft.ui.InlineButton?, which is another preset like LargeButton? and BlockButton?. This time icon only and no label, made to fit into rftLineWithActions widgets. highlightColor: White/Black? allows for two different versions depending on icon background.
• Make copies of rft/ui/LineWithActionsWidget and rft/ui/Selector, with -Thijs appended to the classnames. These are an attempt at integrating the newly created buttons in their intended contexts. The original classes and templates are untouched.
• Changed LargeButton? and BlockButton? to be less hardcoded and just better.

Location:

Dev/branches/rest-dojo-ui/client/rft/ui

Files:

• BlockButton.js (modified) (2 diffs)
• InlineButton.js (added)
• LargeButton.js (modified) (2 diffs)
• LineWithActionsWidgetThijs.js (added)
• SelectorThijs.js (added)
• templates/LineWithActionsWidgetThijs.html (added)
• templates/SelectorThijs.html (added)

Dev/branches/rest-dojo-ui/client/rft/ui/BlockButton.js

@@ -7 +7 @@
             showLabel: true,    // Or false? Documentation is retarded about this boolean... true = no label???
             highlightColor: 'blue', // Use the CSS class "highlightBlue" this adds to style active/hover transitions.
+            baseClass: "rftBlockButton",
             postMixInProperties: function(){
                 if (this.iconType == "none" || this.iconType == null) {
@@ -17 +18 @@
             postCreate: function(){
                 var capitalizedHighlightColor = this.highlightColor.charAt(0).toUpperCase() + this.highlightColor.slice(1);
-                domClass.add(this.domNode, "rftBlockButton highlight"+capitalizedHighlightColor);
+                domClass.add(this.domNode, "highlight"+capitalizedHighlightColor);
                 this.inherited(arguments);
             }

Dev/branches/rest-dojo-ui/client/rft/ui/LargeButton.js

@@ -7 +7 @@
             showLabel: true,    // Or false? Documentation is retarded about this boolean... true = no label???
             highlightColor: "blue", // Use the CSS class "highlightBlue" this adds to style active/hover transitions
+            baseClass: "rftLargeButton",
             postMixInProperties: function(){
                 this.iconClass = "rftIcon rftIcon"+this.iconType;
@@ -13 +14 @@
             postCreate: function(){
                 var capitalizedHighlightColor = this.highlightColor.charAt(0).toUpperCase() + this.highlightColor.slice(1);
-                domClass.add(this.domNode, "rftLargeButton highlight"+capitalizedHighlightColor);
+                domClass.add(this.domNode, "highlight"+capitalizedHighlightColor);
                 this.inherited(arguments);
             }