#593: Output collection in body classes for collection based css styling
--------------------------+----------------------
Reporter: tbrooks | Owner: tbrooks
Type: enhancement | Status: in_merge
Priority: minor | Milestone:
Component: WebSearch | Version:
Resolution: | Keywords: INSPIRE
--------------------------+----------------------
Comment (by jcaffaro):
Replying to [comment:4 tbrooks]:
> I've just amended and re-pushed this to account for this, I simply used
the function you discussed to wash the collection name.
\\
I am not sure that {{{nmtoken_from_string()}}} will be enough in general:
it would for eg. not prevent that the resulting CSS class name does not
start with a digit (which should be forbidden for a CSS class name). Note
that I have not read the CSS specifications to check which other invalid
cases might occur....
\\
> I did run kwalitee and fixed a few minor issues, but didn't make an
attempt on any of the major issues that appear to predate my patch. Did
I miss something?
\\
No, I was just picky about the space before/after the "=" operator ;-) :
{{{
body_css_classes=[]
^
}}}
--
Ticket URL: <http://invenio-software.org/ticket/593#comment:5>
Invenio <http://invenio-software.org>
