This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.
| Summary: | Problems with CSS padding | ||
|---|---|---|---|
| Product: | obsolete | Reporter: | _ krystyna <krystyna> |
| Component: | visualweb | Assignee: | Peter Zavadsky <pzavadsky> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | blocker | ||
| Priority: | P4 | ||
| Version: | 6.x | ||
| Hardware: | Other | ||
| OS: | All | ||
| Issue Type: | DEFECT | Exception Reporter: | |
| Attachments: |
Test project
adjusted project with the Page2 better showing the problem |
||
|
Description
_ krystyna
2007-10-09 03:23:31 UTC
Created attachment 50460 [details]
Test project
Yes, it is there, I need to have a closer look at it. Also here the html was copied into webjsfui:form, that is incorrect, I adjusted the example. The first problem seems to be here, the size of the font used in the div. In the designer the batik parses the size to 1.0, which makes the line virtually invisible. But the browser shows it larger, looking into it. In the example there is a mistake in the style, it provides: font: 20px/1 Ahem it should be: font: 20px Ahem After fixing that, one can see the problem, that the first row is not green as expected. Created attachment 50639 [details]
adjusted project with the Page2 better showing the problem
Also discovered issue with refreshing (the transient style were kept), fixing that part first: Checking in visualweb/insync/src/org/netbeans/modules/visualweb/insync/models/FacesModel.java; /cvs/visualweb/insync/src/org/netbeans/modules/visualweb/insync/models/FacesModel.java,v <-- FacesModel.java new revision: 1.23; previous revision: 1.22 done The construction : font: 20px/1 Ahem is valid and the "1" refers to <line-height> You changed it because we do not support that? Now we can see that the padding does continue to the previous row to overlap the letter. Currently this implementation doesn't count with this kind of overlap. I also think this case is very minor, and very unusual corner case. Please, downgrade this. Krys, where did you find that construct is valid, I was trying to look into the spec, but didn't find anything like that. It might be valid, I just don't know about that. In any case, the parser which we use (batik) doesn't support it. Also that would be a separate issue. This issue is about that current implementation layout doesn't take into account padding, borders, margins of parts of texts in line (span typically). *** Issue 118274 has been marked as a duplicate of this issue. *** *** Issue 118296 has been marked as a duplicate of this issue. *** Peter,CSS2.1 spec, search at: 15.8 Shorthand font property: the 'font' property http://www.w3.org/TR/CSS21/fonts.html Downgrading Yes, the font thing is there, please, file a separate issue for that one (low priority please). There is no current plan to resolve it. Passing to 'later'. |