Additional CSS class for version status
|Assignee:||Jean-Philippe Lang||% Done:|
The version status (open, locked, closed) isn't accessible for theme designers. Neither in Roadmap view nor in Version details view.
This patch adds another CSS class after #19341 in Roadmap view.
Plus it adds the new CSS classes from #19341 and the ones introduced here also in Version details view.
It can be discussed where to place the CSS classes. I put it on
<div id="#roadmap"> but I believe it could also be nice to have them available when styling the
<h2> heading... More opinions?
#1 Updated by Tobias Fischer over 2 years ago
Just for the record: New CSS classes are:
version-status-openfor open versions
version-status-lockedfor locked versions
version-status-closedfor closed versions
This information is more useful than "completed/incompleted" from #19341 when trying to add another version icon for locked versions.
#3 Updated by Tobias Fischer over 2 years ago
Jean-Philippe Lang wrote:
Yes. However, I wasn't sure where to add those classes:
Sure, can you update your patch?
- I assume
<body>isn't appropriate as we normally don't have such classes there.
- I thought about an
<article>tag around the whole content in
app/views/versions/show.html.erbto match the behaviour from patch #19341
- or to add those classes to the
<div id="content">wrapper (which seems to be uncommon as well, right?)