Ticket #4408 (new enhancement)

Opened 10 months ago

Last modified 5 months ago

[patch] [CLA] Form widget: process native elements in addition to widgets

Reported by: guest Owned by: wolfram
Priority: normal Milestone: 1.2
Component: Dijit Version: 0.9
Severity: normal Keywords:
Cc: gizmojo.org, wolfram.kriesing@…

Description

dijit.form.Form.getValues() currently ONLY submits values of widgets inside the form not values of normal form elements, this patch fixes that.

Attachments

form-getValues-fix.diff (1.0 kB) - added by guest 10 months ago.
temporary simple fix, to make it at least work

Change History

Changed 10 months ago by guest

temporary simple fix, to make it at least work

Changed 10 months ago by guest

for the record I proposed the patch wolfram.kriesing@…, guest might not have a CLA :-)

Changed 10 months ago by bill

  • owner deleted
  • summary changed from dijit.form.Form.getValues() temporary fix to [patch] [CLA] dijit.form.Form.getValues() temporary fix
  • component changed from General to Dijit
  • milestone set to 1.1

Changed 10 months ago by guest

Set to milestone 1.1? Why that? If the form elements that are no widgets dont get returned by getValues() the widget will be unusable for those forms not using widgets for all the form elements, as for me for example. In 0.4 that was very well possible. Is my provided quick-fix breaking something that it could not be applied before 1.1?

Changed 10 months ago by bill

  • type changed from defect to enhancement
  • summary changed from [patch] [CLA] dijit.form.Form.getValues() temporary fix to [patch] [CLA] Form widget: process native elements in addition to widgets

Patch looks good but also, the set function needs to handle native elements (in addition to the get function), and test file needs to be updated to check that.

Thanks!

Changed 10 months ago by gizmojo.org

  • cc gizmojo.org added

Changed 9 months ago by bill

  • owner set to wolfram

Hi Wolfram,

Can you update this patch so that:

  • checkboxes and radio buttons work
  • update test_Form.html to have native elements in addition to widgets
  • make setValues() work too

Otherwise we have to say that Form sort-of works for native elements, which I guess is better than nothing, but not ideal.

Thanks!

Changed 7 months ago by dylan

  • cc wolfram.kriesing@… added

Wolfram, please see bill's previous comments... I think getting this fixed for Dojo 1.1 is important.

Changed 6 months ago by wolfram

i will try to get it done for the 1.1beta

Changed 6 months ago by dante

(In [12012]) refs #4408, #5359, #5550 - adds semantic degradable textarea, select, and hidden input types that work with dijit.form.Form

Changed 5 months ago by peller

  • milestone changed from 1.1 to 1.2
Note: See TracTickets for help on using tickets.