Opened 9 years ago

Closed 9 years ago

#11549 closed enhancement (wontfix)

InlineEditBox: Allow save default value

Reported by: Martin Grames Owned by:
Priority: high Milestone: tbd
Component: Dijit Version: 1.5
Keywords: Cc:
Blocked By: Blocking:

Description

InlineEditBox? doesn't allow save default value, because intermediateChanges can't set via parametr.

Possible solution:

InlineEditBox.js, line 444:
from:
if("intermediateChanges" in cls.prototype){
to:
if(this.inlineEditBox.intermediateChanges){

Usage:

this.editor = new dijit.InlineEditBox({
    autoSave: false,
    intermediateChanges: false,
    // or
    //intermediateChanges: true
    // ...
}, editSpan);

Change History (6)

comment:1 Changed 9 years ago by bill

Can you explain what you mean by "default value"?

comment:2 in reply to:  1 Changed 9 years ago by Martin Grames

editSpan from example:

var editSpan = document.createElement('span');
editSpan.appendChild(document.createTextNode(myText));

myText from span is default value for InlineEditBox?, for this value is button Save disabled - I need this button active. That's why I want to set variable intermediateChanges through the parameter.

comment:3 Changed 9 years ago by bill

You want to let the user press save even though they aren't changing the value of the InlineEditBox (iow, the new value is the same as the old value)?

comment:4 Changed 9 years ago by Martin Grames

Yes, I have multiuser application, on the page is menu with directories and items, and every user can add own directory to menu. After click to new directory button is added to menu InlineEditBox? with username as default value - this value can user modify or save without changes.

comment:5 Changed 9 years ago by bill

Type: defectenhancement

Hmm, it's just that clicking an InlineEditBox to edit the value and then pressing save, without changing the value, seems meaningless.

It doesn't sound like an issue with intermediateChanges, since that just affects whether or not the InlineEditBox is notified of the changed value on every keystroke or not.

comment:6 Changed 9 years ago by bill

Resolution: wontfix
Status: newclosed
Note: See TracTickets for help on using tickets.