Skip to content

NLR updates and improve formatting#76

Merged
jd-lara merged 11 commits into
mainfrom
kd/nlr
Apr 28, 2026
Merged

NLR updates and improve formatting#76
jd-lara merged 11 commits into
mainfrom
kd/nlr

Conversation

@kdayday

@kdayday kdayday commented Apr 22, 2026

Copy link
Copy Markdown
Collaborator

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Sienna website content and styling to reflect “NLR” branding, refresh homepage visuals, and tighten header/footer layout consistency.

Changes:

  • Replaced NREL references/URLs/emails with NLR equivalents across homepage, nav, resources, and footer.
  • Updated homepage hero/service copy and swapped the “about” image asset.
  • Adjusted site CSS for header/footer spacing, typography, and testimonial color styling.

Reviewed changes

Copilot reviewed 10 out of 13 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pages/resources.md Updates “Work with Us” contact email to the new domain.
pages/publications.md Updates PR100 publication venue/URL to NLR branding.
assets/img/nlr.png Adds new “about” section image used on the homepage.
assets/css/main.min.css Tweaks header/footer/banner spacing + typography; adds testimonial styling.
_layouts/home-sienna.html Updates homepage email link, copy, and “about NLR” section + image/URL.
_includes/footer/footer-sienna.html Updates footer branding, layout columns, and contact details.
_data/testimonials/testimonials-sienna.yml Updates testimonial company strings from NREL → NLR.
_data/settings.yml Updates canonical contact email/mailto values.
_data/navigation-menu/sidemenu.yml Updates ABOUT US external link to nlr.gov.
_data/navigation-menu/secondary.yml Updates ABOUT US external link to nlr.gov.
README.md Adds local dev note about jekyll clean and cache/CSS visibility.
Comments suppressed due to low confidence (2)

_includes/footer/footer-sienna.html:21

  • The footer hard-codes a phone number that doesn't match the canonical contact info in _data/settings.yml (contact_settings.phone_number). To avoid inconsistent contact details across the site, either update the settings value to match or render the footer phone number from the settings data.
            <li>
              <span class="wz-icon"><i class="lni lni-phone"></i></span>
              <p>(+1) 303-275-3921</p>
            </li>

assets/css/main.min.css:19077

  • The selector .nav-toggle-btn.white.black-on-mobile .stick now sets the stick color to white, which conflicts with the intent implied by the black-on-mobile class name and also diverges from the toggle title color (still forced to black below). This is likely to cause visibility/contrast issues on mobile; either revert the stick color to black for this variant or rename/adjust the class + related styles so behavior matches the naming and the title/sticks are consistent.
    .nav-toggle-btn.white.black-on-mobile .stick {
        background-color: #fff !important;
    }

    .nav-toggle-btn.white.black-on-mobile .nav-toggle-title {
        color: #000 !important;

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pages/publications.md Outdated
Comment thread _includes/footer/footer-sienna.html Outdated
@kdayday kdayday requested a review from jd-lara April 22, 2026 20:14
@jd-lara jd-lara merged commit d91b690 into main Apr 28, 2026
1 check passed
@jd-lara jd-lara deleted the kd/nlr branch April 28, 2026 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants