[ 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.