admin/settings/sifr has inline styles on both the version and the rules table. Inline styles are a pain for themers (especially the clear: both on the table) so I've removed them and added two classes; .sifr-version and .sifr-rules, allowing themers to take care of the styling themselves.

CommentFileSizeAuthor
sifr_class_additions.patch1.43 KBgarrettc

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: Inline styles on admin/settings/sifr » Hard-coded CSS in rules overview
Project: sIFR » Dynamic Rendering
Version: 5.x-1.2 » 5.x-1.x-dev
Status: Needs review » Needs work

You missed to add the removed styles to the module's css file.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs work » Fixed
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)