| Summary: | [Theming] Border support not consistent | ||
|---|---|---|---|
| Product: | [RT] RAP | Reporter: | Stefan Röck <stefan.roeck> |
| Component: | RWT | Assignee: | Project Inbox <rap-inbox> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | P3 | CC: | gunnar, Istvan.Ballok |
| Version: | 1.4 | ||
| Target Milestone: | 3.0 M2 | ||
| Hardware: | PC | ||
| OS: | All | ||
| Whiteboard: | |||
The properties "border-bottom", "border-top", etc. are not supported by most widgets, only the shorthand "border". We support only those properties that are listed in the *.theme.xml (and also listed in the theming reference).
You could rewrite your code like this:
Button[PUSH] {
border: 1px 0 1px 0; /* top right bottom left */
}
or, in the short form:
Button[PUSH] {
border: 1px 0; /* top/bottom right/left */
}
If you are requesting support for common individual properties AND shorthands, please change the summary accordingly. This would affect all widgets, not only Button.
Hi Ralf, I just tried what you suggested and got the following error: org.w3c.css.sac.CSSException: Failed to read property border: Failed to parse color 0 DIM 1.0px 0 I am using RAP 1.3 Maintenance Branch - What's wrong here? Hi Stefan, it's right, you have to replace "0" with "0px" in RAP. In CSS, you can omit the unit for zero values. This doesn't work in RAP. I opened bug 324820. Hold on, I was completely misdirected. My snippet was non-sense, I apologize. What I wrote works for margin and padding, but not for borders. It would work for border-width, but this property is not yet supported. We currently cannot deal with different border widths for the different edges. See bug 283872. Four border edges are supported since RAP 3.0 M2. |
It would be nice if the most common css border definitions are supported. In the following example, border-bottom works, but border-top throws an error ("unknown property") Button[PUSH] { border-bottom: 1px; border-top : 1px; }