Hidden field problem

[Garry Yao]

There were many problems with hidden fields.

Reproducing Procedures

1. Open the "replace by class" page, insert a form;
2. Insert a hidden field with all attributes specified
   • Actual Result in Firefox: The inserted is not visible.
3. Right click the hidden field to open context menu
   • Actual Result: The hidden field property entry is missing
4. Click to open the hidden field dialog again;
   • Actual Result: All the attributes of it are lost.

[Garry Yao]

Accumulate Firefox problem.

[Frederico Caldeira Knabben]

We should have something similar to the V2 solution here.

[Matti Järvinen]

FF doesn't show context menu for select, radio and checkbox and those cannot be selected by clicking either.

Fix in V2 replaces the checkbox, radio button and select box with clickable placeholders.

Could you give me a short description on how to make fake elements with CKEditor and how to use them? Pseudo code example will do (just to get me on the right track) so I can asses how much work would go into fixing this.

[Frederico Caldeira Knabben]

Replying to matti:

Could you give me a short description on how to make fake elements with CKEditor and how to use them? Pseudo code example will do (just to get me on the right track) so I can asses how much work would go into fixing this.

Take a look at the "link" plugin, which creates fake elements for anchors. It should give you some hints on this.

[Matti Järvinen]

I don't have svn version at hand so no patches here only changed files. Fixes radio, checkbox and hiddenfield with fake elements. Images are from V2 fix.

Few questions:

• Is creation of new html element required every time ( Always create a new anchor, because of IE BUG.)?
  
• How could this be redisigned to use original field commit methods?
  
• Any thoughts on select element, how that should be fixed for FF?

Comments please. We really would appreciate this to be fixed in next release.

[Matti Järvinen]

Form plugin changed files for comment

[Frederico Caldeira Knabben]

We should have a fake element for hidden fields (V2) (@matti, I didn't look at your changes).

[Matti Järvinen]

@fredck I used fake element for hidden field.

[Alfonso Martínez de Lizarrondo]

#4965 has been marked as dup

[Minh Nguyen]

Thank matti.

[Matti Järvinen]

I'd would prefer to see someone fix this so that it uses setup and commit (easier to modify if needed for some reason).

Notice that my original zip has fixes for radio and checkbox too (which cannot be modified once created atleast on FF3.6).

[Garry Yao]

Replying to matti:

I'd would prefer to see someone fix this so that it uses setup and commit (easier to modify if needed for some reason).

I can't see how this could improve maintainability.

Notice that my original zip has fixes for radio and checkbox too (which cannot be modified once created atleast on FF3.6).

We'll have a new ticket to discuss the trade-off to apply this also to checkbox etc.

[Garry Yao]

Replying to m.nguyen: ​More efforts is needed to make it work with editor during I/O.

[Matti Järvinen]

Replying to garry.yao:

I can't see how this could improve maintainability.

Sorry I should have been more precise what I ment was that patch doesn't use commit and setup functions of dialog definition and it makes it hard to add more dialog properties for the input by extending if needed.

For example we use additional required attribute for checkbox and some other inputs ( that is parsed within our cms when displaying, posting and receiving the form ). Without dialogdefinition.contents[].commit and setup functions we have to overwrite all of the onOK function to achieve this.

Replying to garry.yao: zip has this on plugin.js line 225 -> 274 though needs some work if radio and checkbox is not handled at the same time.

[Garry Yao]

Replying to matti:

Replying to garry.yao:

I can't see how this could improve maintainability.

Sorry I should have been more precise what I ment was that patch doesn't use commit and setup functions of dialog definition and it makes it hard to add more dialog properties for the input by extending if needed.

I got your points, it would be better if fields commit directly to the real element while the fake elements are created from it at end, transparently from the user.

[Minh Nguyen]

For hiddenfield

[Garry Yao]

Please cleaning up coding styles and make sure the fake element image file is committed, also note that the commit should fall into to 3.2.x branch instead of trunk.

[Garry Yao]

Replying to garry.yao:

Please cleaning up coding styles and make sure the fake element image file is committed, also note that the commit should fall into to 3.2.x branch instead of trunk.

Typo: 3.3.x branch.

[Minh Nguyen]

Fixed with [5295].