Conversation
Added fields for specifying id and class for external CSS Added extra validation Most fields are not required Works great with bootstrap!
|
Actually, the plugin package is intended to be lightweight and to have as less dependencies as possible. So I've dropped |
|
Please also note, that fields defaults are chosen that they reflect ones defined in W3 CSS spec. This is intentional. |
|
Hi Alex, Well I don't think you gain anything by not using sekizai since as you say it is already required by the CMS. Plus having inline style tags is not valid HTML4 - CSS must go in the head section. Also the main use of this plugin will be to override styles already defined in included CSS, that's why the tag should be blank by default. Hope that makes sense! |
|
Hi Richard, I've incorporated most of the patch you submitted. I think, after some testing there will be a new release in October. Thanks for contributing :) |
|
Hi Alex, no worries. I'll have a merge and see where we are. Yes forgot CSS has an inherit option, not specified makes more sense. |
|
Oops, my fault, missed that paragraph about enforced sekizai namespaces :) |
I've had a rethink about this and decided the original method of wrapping a placeholder is more flexible. However I have made some useful enhancements.
I have only tested on my python 3.5.2 django 1.9.9 environment.