On a second thought, I think we shouldn't name it Panel but ExtJS::Panel. And we should add a corresponding HTML::FormFu::Element::ExtJS::Panel which renders the items regulary.

When I wrote FormFu::ExtJS I had always in mind that it should work, even if we remove the ::ExtJS from new(). It would be great if this would still work, even if we add a Panel object.


Thanks for yur efforts anyway!

Moritz


Am 24.03.2009 um 13:28 schrieb Moritz Onken:

I think we should make Fieldset a subclass of your Panel class.

Just copy the code from Fieldset and put in your code and name it Panel and create new class Fieldset which sublasses from Panel.


moritz


Am 24.03.2009 um 15:07 schrieb Mario Minati:

Hi Moritz,

I'm wondering if it would be a better idea to make the xtype of fieldset
configurable as a panel might be seen as fieldset.

I would like to create a class that can handle more than just a plain panel. Currently I'm useing accordion and tabpanel and still have some custom Ext panel like classes like Columnform for setting fixed width of fields and an
Expandable Fieldset.

The new panel class can set all configuration properties as attributes like
so:
- type: Panel
  xtype:     panel
  label_loc: Company
  attributes_xml:
    iconCls:    panel-icon-company
    autoScroll: true
    layout:     border
    border:     false
    frame:      true
  elements:

But that would also be possible with the current fieldset class if we make
xtype configurable.

What do you think?


Greets,

Mario Minati

Am Dienstag 24 März 2009 10:53:41 schrieb Moritz Onken:
Sounds good to me. Please make sure that tests are present.
Finally TabPanels come to FormFu::ExtJS :-)

Moritz

Am 24.03.2009 um 03:34 schrieb Mario Minati:
Currently I don't see any element which renders as a ExtJS panel.

So I'm creating one, which can have a given xtype and has children
elements.

As there is no use of such an element in HTML FormFu I would put the
appropriate HTML::FormFu::Element::Panel class also into the ExtJS
branch.

Would that be ok?


Greets,

Mario Minati

_______________________________________________
HTML-FormFu mailing list
HTML-FormFu@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/html-formfu

_______________________________________________
HTML-FormFu mailing list
HTML-FormFu@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/html-formfu



_______________________________________________
HTML-FormFu mailing list
HTML-FormFu@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/html-formfu


_______________________________________________
HTML-FormFu mailing list
HTML-FormFu@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/html-formfu


_______________________________________________
HTML-FormFu mailing list
HTML-FormFu@lists.scsys.co.uk
http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/html-formfu

Reply via email to