Adding support for editable field on info window

Description

Currently info window can run process, But there is no way to make any of field as editable by user.

Adding this can allow to create suggestion and then user corrected input for process

Environment

None

Activity

Show:
FreePath
September 25, 2018, 10:40 AM
Edited

Hi muriloht , thanks for the test !
Your analysis is correct, the "index offset" solution is inadegaute when there are hidden columns. I rewrote that part so that now the renderer has the correct information about what columns are displayed.

Please See 2709-180925.patch for the fix.I strongly suggest applying too.

Best regards,
Silvano

Hiep Lq
September 27, 2018, 2:15 AM
Edited

WListbox.allowIDColumnForReadWrite should default is true.
because almost WListbox is allow choose (example copy from at material receipt)

allowIDColumnForReadWrite don't use anywhere. it's necessary or miss patch?

Hiep Lq
September 27, 2018, 2:47 AM

WListbox.isCellEditable
always false when m_readWriteColumn.contains(new Integer(column)) = false
it's correct?

FreePath
September 27, 2018, 7:09 AM

Hi,
the whole isCellEditable function is broken because of (reported as a new issuse since that one was closed).
It does not default to true because the previuos behaviour did not allow IDColumn (just booleans) to be used as flag for cell editability, so i kept backward compatibility to avoid breaking current functionalities.

Best regards,
Silvano

Carlos Ruiz
September 28, 2018, 3:42 PM

Assignee

Deepak Pansheriya

Reporter

Deepak Pansheriya

Tested By

None

Components

Priority

Major
Configure