Skip to content

Conversation

matt-bernhardt
Copy link
Member

@matt-bernhardt matt-bernhardt commented Apr 21, 2023

Why are these changes being introduced:

The set of parent theme styles was apparently created without those which apply to the 2021 location template.

Relevant ticket(s):

https://mitlibraries.atlassian.net/browse/lm-305

How does this address that need:

This brings up the block of styles for that template.

Document any side effects to this change:

None for this change, but I'm trying to understand how these styles (and some other more recent changes to the theme) were missed during the initial theme build-out. I wonder whether my copy of the parent theme was somehow behind the main branch at the time.

Developer

Secrets

  • No secrets are affected

Documentation

  • No documentation changes are needed

Accessibility

  • ANDI or Wave has been run in accordance to
    our guide and
    all issues introduced by these changes have been resolved or opened as new
    issues (link to those issues in the Pull Request details above)

Stakeholder approval

  • Stakeholder approval has been confirmed

Dependencies

NO dependencies are updated

Code Reviewer

  • The commit message is clear and follows our guidelines
    (not just this pull request message)
  • The changes have been verified
  • The documentation has been updated or is unnecessary
  • New dependencies are appropriate or there were no changes

** Why are these changes being introduced:

* The set of parent theme styles was apparently created without those
  which apply to the 2021 location template.

** Relevant ticket(s):

* https://mitlibraries.atlassian.net/browse/lm-305

** How does this address that need:

* This brings up the block of styles for that template.

** Document any side effects to this change:

* None for this change, but I'm trying to understand how these styles
  (and some other more recent changes to the theme) were missed during
  the initial theme build-out. I wonder whether my copy of the parent
  theme was somehow behind the main branch at the time.
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.

2 participants