Skip to content

base.njk uses site-header/site-footer class names instead of header/footer #3

Description

@MelbourneDeveloper

Problem

The base.njk template uses class="site-header" and class="site-footer" for the header and footer elements. Standard convention and most CSS frameworks use simply class="header" and class="footer".

Current behavior

<header class="site-header">
<footer class="site-footer">

Expected behavior

<header class="header">
<footer class="footer">

Impact

Sites migrating to the plugin that use .header and .footer selectors in CSS and tests need to change all references to .site-header and .site-footer.

The dart_node site uses .header and .footer throughout its CSS and test suite.

Suggestion

Use class="header" and class="footer" to match standard conventions. The <header> and <footer> HTML5 elements already provide semantic meaning, so the site- prefix is redundant.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions