[ 
https://issues.apache.org/jira/browse/OFBIZ-1648?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12569724#action_12569724
 ] 

Adrian Crum commented on OFBIZ-1648:
------------------------------------

David,

Thank you for taking the time to look at this.

The problems I've had in the past all boil down to the fact that the various 
widgets did not derive from a common class, so it is difficult to add 
functionality to the widget library as a whole without having to write a lot of 
redundant code.

Why would I want to write four or five versions of the boundary comment code - 
one for each widget? It is far simpler to write it once in the base class.

The problem I have with the widgets right now is that the new screenlet widget 
can't interoperate with any widgets that aren't direct descendants of the 
screenlet widget. In other words, it can do things with its own subelements, 
but not subelements contained within its subelements. This patch (once 
completed) will fix that.

I don't see how a "direct" approach would be better than simple OOP code. I'm 
not sure I know what a "direct" approach is.


> Sandbox: Improved Screen Widgets, adding 3rd party rendering library support
> ----------------------------------------------------------------------------
>
>                 Key: OFBIZ-1648
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1648
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>            Reporter: Adrian Crum
>            Priority: Minor
>         Attachments: widget_improvements.patch
>
>
> Discuss screen widget modifications and enhancements that will make the 
> screen widgets more robust. Everyone is welcome to review and comment.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to