| Summary: | [testcase] Invalid css padding declaration should not be rendered | ||
|---|---|---|---|
| Product: | [Applications] konqueror | Reporter: | Andrea Iacovitti <aiacovitti> |
| Component: | khtml | Assignee: | Konqueror Developers <konq-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | NOR | ||
| Version: | 4.13.2 | ||
| Target Milestone: | --- | ||
| Platform: | unspecified | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/kdelibs/bb2e6d6ebf8daf2d8fc838c5582204c7cfc3d1a6 | Version Fixed In: | 4.13.3 |
| Attachments: | self explanatory testcase | ||
Git commit bb2e6d6ebf8daf2d8fc838c5582204c7cfc3d1a6 by Andrea Iacovitti. Committed on 10/06/2014 at 14:34. Pushed by aiacovitti into branch 'KDE/4.13'. Remove subproperties previously added by parseValue() to our 'parsedProperties' if the shorthand is invalid. FIXED-IN: 4.13.3 M +23 -4 khtml/css/cssparser.cpp M +1 -0 khtml/css/cssparser.h http://commits.kde.org/kdelibs/bb2e6d6ebf8daf2d8fc838c5582204c7cfc3d1a6 |
Created attachment 87105 [details] self explanatory testcase The following padding shorthand declaration: padding: 20px 20px boo is invalid, thus it should be treated the same way as 'padding: 0px'. khtml instead applies a padding-top/padding-right of 20px.