[
https://issues.apache.org/jira/browse/MSKINS-244?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17825262#comment-17825262
]
Konrad Windszus commented on MSKINS-244:
----------------------------------------
There seems to be a difference between those classes, though:
# {{table}} is set via
https://github.com/apache/maven-fluido-skin/blob/d5eef2752bdb2e3d395438a502ab87ecbebd9e8c/src/main/resources/META-INF/maven/site.vm#L242
(so no need for the Doxia Site to generate those, i.e. a fluido internal class.
# {{verbatim}} needs to be set by Doxia, is not added through any Fluido
Velocity logic.
> Get rid of CSS classes "table" and "verbatim"
> ---------------------------------------------
>
> Key: MSKINS-244
> URL: https://issues.apache.org/jira/browse/MSKINS-244
> Project: Maven Skins
> Issue Type: Improvement
> Components: Fluido Skin
> Reporter: Konrad Windszus
> Priority: Major
>
> Both classes {{table}} and {{verbatim}} feel redundant to me, as for both we
> do have a semantic element (with the same meaning) which is {{<table>}} and
> {{<pre>}}.
> Therefore the CSS referring to those should be removed:
> https://github.com/apache/maven-fluido-skin/blob/d5eef2752bdb2e3d395438a502ab87ecbebd9e8c/src/main/resources/css/maven-base.css#L32-L33
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
