Jacopo,

Thanks for your nice reply.
First I read Subject of Issue and that was  "Replace all div
class="tabletext" wrappers with <p> or just remove them and set the style
info in the BODY element in the css file "

So I thought to change the div class="tabletext" into p's.
I am sorry for not reading and thinking your details properly and that was
"all <div class="tabletext"> wrappers can be removed, the exception being
where they are used like <p>s - the default text style can be set on body;"

And now from your's and chris comments I got the idea of what to do.
I will submit the patch for this task whenever I will get time.

Thanks Jacopo & Chris for your support and once again sorry for my stupid
mistake.


On 1/6/07, Jacopo Cappellato (JIRA) <[EMAIL PROTECTED]> wrote:


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

Jacopo Cappellato commented on OFBIZ-588:
-----------------------------------------

Ashish,

yes Chris is right (and sorry for not being very clear about this task);
however I'm reviewing the patch for the accounting component and it looks
you did the right thing, except for the <div> elements inside table cells
<td> that should be removed and not replaced by <p> tags.
So for example:

<td><div class="tabletext">some text...</div></td>
should become
<td>some text...</td>

and
<td><div class="tabletext"><b>ID</b></div></td>
should probably become
<th>ID</th>
(to all: is it correct?)

Thanks for your great help!



--
Regards
Ashish Vijaywargiya
Aditisoft Technology Laboratory

Reply via email to