Opened 15 years ago
Closed 15 years ago
#819 closed defect (fixed)
Calling 'disable()' has no effect on InlineEditBox.
Reported by: | Ashish | Owned by: | alex |
---|---|---|---|
Priority: | high | Milestone: | 0.9 |
Component: | Widgets | Version: | 0.2 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Calling 'disable()' has no effect on InlineEditBox?.It seems enable/disable functions have not been over-riden for this control.
Change History (4)
comment:1 Changed 15 years ago by
Milestone: | → 0.5 |
---|
comment:2 Changed 15 years ago by
Owner: | changed from Alex to alex |
---|
comment:3 Changed 15 years ago by
comment:4 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
(In [5409]) Applying changes to give InlineEditBox? a working enable/disable implementation. Fixes #819
Note: See
TracTickets for help on using
tickets.
Here is the patch to allow for eanbling and disabling of the InlineEditBox? in both editing mode and non-editing mode:
Index: InlineEditBox?.js =================================================================== --- InlineEditBox?.js (revision 5408) +++ InlineEditBox?.js (working copy) @@ -34,6 +34,7 @@
+ disabled: false,
@@ -54,7 +55,7 @@
+ if(!this.editing && !this.disabled){
@@ -72,7 +73,7 @@
@@ -119,6 +120,22 @@
+ disable: function(e){ + this.submitButton.disabled = true; + this.cancelButton.disabled = true; + var ee = this[this.mode.toLowerCase()]; + ee.disabled = true; + this.disabled = true; + }, + + enable: function(e){ + this.checkForValueChange(); + this.cancelButton.disabled = false; + var ee = this[this.mode.toLowerCase()]; + ee.disabled = false; + this.disabled = false; + }, +