On Monday 09 February 2009, Dan Korostelev wrote:
> FileWidget - It doesn't clear the bytes value if no new file is
> uploaded now, which is nice. But there's also should be a way to clear
> current value if the field is not required. I've added that to the
> TODOS.txt. I think that should be done before release to make the
> widget actually functional out of box.

Since this feature has not been there before, I can live without it for the 
2.0 release.

> MultiWidget - Probably needs some review as it does the updateWidgets
> thing on "value" property setting, which seems fishy to me.

Me too. :-) Roger, could you comment on this? Or Adam?

> It works 
> however. I've added the check for min_length and max_length and
> conditional buttons in the browser version. One thing I'd like to see
> is that it generate a default number of input rows based on field's
> minimal length if there's any. Also, another thing that would be nice
> is to have a way to reorder values for orderable fields. However this
> can wait for next release. I've added that to the TODOS.txt.

Nice to have, but not necessary. ;-)

> ObjectWidget/ObjectMultiWidget - ??? I didn't checked that out, so it
> would be nice if its author reviewied it and wrote here about its
> status.

Adam? I'll note that we use that code heavily at Keas, so at least for that 
limited exposure it seems fine.

> Source support - Seems to work fine. I've checked that out in my
> sandbox instance with zc.sourcefactory's context-less and
> context-based sources.

That's great to hear.

> However, there was some backward-incompatible 
> refactorings (class renames) done to sequence data converters that
> breaks the z3c.pt benchmarking suite. This may also break end-users'
> code so we probably want to fix the compatibility.

Yeah, let's fix that.

> z3c.pt support - ??? I don't use the z3c.pt myself, so I didn't really
> checked that out. As I said before, the benchmarking suite is borked.
> Also, there's a strange bug with macros (see below). Also, we need to
> migrate to z3c.ptcompat instead of z3c.pt.compat (UPDATE: the merge
> was done while I was writing this).

Malthe, do you have some time to look into this?

> Tests - All are passing.

Clearly, all testsshould be passing. In addition, I would really like to see 
100% test coverage after taking the false positives into consideration.

> Translations - I've updated the template and the russian translation
> to be complete. I don't expect any msgid changes, so I think
> translators should review their translations and commit them right now

If translations are not updated until the next release, 2.1.0 or 2.0.1, that's 
fine with me.

> Also, I wanted to add browser widget attributes like "klass" or
> "onclick" to adaptable values. This will require some work to make
> browser widgets automatically add their adaptable values to
> _adapterValueAttributes before calling parent's "update" method. I'm
> not sure that I'll be doing that very soon as it isn't very important.
> So this is definetely not a reason to wait with the release.
> One more thing I'd like to do is to add "klass" and "id" to the forms
> themselves so one could easily customize the visual appeal of the
> forms. But it's probably should be done in z3c.formui's subclasses and
> not in z3c.form's base classes. I'd like to hear the community opinion
> on that though.

All nice to have. :-) I would not block the release because of it. 

Stephan Richter
Web Software Design, Development and Training
Google me. "Zope Stephan Richter"
Zope-Dev maillist  -  Zope-Dev@zope.org
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope )

Reply via email to