You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Adam Heath (JIRA)" <ji...@apache.org> on 2010/04/18 03:05:26 UTC
[jira] Issue Comment Edited: (OFBIZ-3708) Fields should be defined
with span not div. (radio, date, lookup)
[ https://issues.apache.org/jira/browse/OFBIZ-3708?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12858249#action_12858249 ]
Adam Heath edited comment on OFBIZ-3708 at 4/17/10 9:05 PM:
------------------------------------------------------------
Sure, div and span are different. There default css properties are different. But you can change that easily enough.
I did review the screenshots. Which is why I knew I could make my comment.
The following html shows a div being displayed inline, no newline before or after. Very simple to do.
<html>
<head>
<style>
div {
border:1px solid red;
}
.span {
display:inline;
}
</style>
</head>
<body>
before
<div class="div">
a div
</div>
middle
<div class="span">
another div
</div>
after
</body>
</html>
was (Author: doogie):
Sure, div and span are different. There default css properties are different. But you can change that easily enough.
I did review the screenshots. Which is why I knew I could make my comment.
<html>
<head>
<style>
div {
border:1px solid red;
}
.span {
display:inline;
}
</style>
</head>
<body>
before
<div class="div">
a div
</div>
middle
<div class="span">
another div
</div>
after
</body>
</html>
> Fields should be defined with span not div. (radio, date, lookup)
> -----------------------------------------------------------------
>
> Key: OFBIZ-3708
> URL: https://issues.apache.org/jira/browse/OFBIZ-3708
> Project: OFBiz
> Issue Type: Bug
> Components: framework
> Affects Versions: SVN trunk
> Reporter: Blas Rodriguez Somoza
> Priority: Minor
> Fix For: SVN trunk
>
> Attachments: CAL_bizznesstime_after.jpg, CAL_bizznesstime_before.jpg, CAL_bluelight_after.jpg, CAL_bluelight_before.jpg, CAL_droppingcrumbs_after.jpg, CAL_droppingcrumbs_before.jpg, CAL_flatgrey_after.jpg, CAL_flatgrey_before.jpg, CAL_tomahawk_after.jpg, CAL_tomahawk_before.jpg, LOOKUP_CAL_bizznesstime_after.jpg, LOOKUP_CAL_bizznesstime_before.jpg, LOOKUP_CAL_bluelight_after.jpg, LOOKUP_CAL_bluelight_before.jpg, LOOKUP_CAL_droppingcrumbs_after.jpg, LOOKUP_CAL_droppingcrumbs_before.jpg, LOOKUP_CAL_flatgrey_after.jpg, LOOKUP_CAL_flatgrey_before.jpg, LOOKUP_CAL_tomahawk_after.jpg, LOOKUP_CAL_tomahawk_before.jpg, OFBIZ-3708_framework_divspan.diff
>
>
> Field markup should be a valid inline element.
> Defining fields with div (block) instead of span (inline) creates several problems:
> - Missalignments in screen. (See LOOKUP_CAL screenshots)
> - Implies a end-of-line after the div. (See CAL screenshots)
> - If used inside inline elements which should be OK, becames XHTML validation errors. (Manufacturing -> MRP -> MRP log)
> Because this is a markup problem, it spans to all the themes.
> This patch contains 2 files in the framework/widget area and 5 in the themes area.
> Because the source of the problem is in the framework/widget area, I opt to assign the bug to the framework area.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira