[ 
https://issues.apache.org/jira/browse/OFBIZ-671?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12475950
 ] 

Adrian Crum commented on OFBIZ-671:
-----------------------------------

I just realized something about the change to the widgets - this could have an 
adverse impact on the eCommerce component. The main issue is browser 
compatibility. 

Some things off the top of my head that need to be addressed for eCommerce:

1. The new styles need to be copied to ecommain.css.
2. The list type form widget defaults to a table style of "basic-table 
dark-grid" - Chris Howe mentioned previously that older versions of IE don't 
handle multiple classes well.
3. The grid table decorators reveal a flaw in IE - it doesn't draw borders 
around empty table cells (<td> elements).

So, those things need to be resolved before this gets committed.


> Remove HTML styling code from widgets
> -------------------------------------
>
>                 Key: OFBIZ-671
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-671
>             Project: OFBiz (The Open for Business Project)
>          Issue Type: Sub-task
>          Components: framework
>            Reporter: Adrian Crum
>            Priority: Minor
>         Attachments: formwidget.patch, formwidget.patch, formwidget.patch, 
> menuwidget.patch, menuwidget.patch, menuwidget.patch, treewidget.patch, 
> treewidget.patch
>
>
> Widgets have embedded styling code in them. Styling should be left to the css 
> files. A CSS class should be created called "alert" or something similar, and 
> that class should be used instead of the hard-coded color="red" found in the 
> widgets.
> Files affected (as far as I can tell): htmlMenuRenderer.java and 
> htmlFormRenderer.java.

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