Note: Some icons were not deleted as they might have a future use (like social media icons). I considered removing all icons unless they are in use but I kept them in case we need a simple icon like a chevron.
Patterns:
All pattern files were removed.
The patterns and patterns/images directories are still in the theme, just empty. This will be the location of patterns when they are added in future projects. The images directory is used by the pattern managers plugin.
The legacy patterns for breadcrumbs and copyright are no longer used.
As the author of this pull request, I verify that:
[x] I have set the target branch to master.
[x] I have detailed the purpose of this Pull Request in a non-technical way.
[x] I have detailed how to test the changes in the Pull Request.
[x] I have detailed the functional tests required for approval.
[x] I have performed a self-review of my code to ensure it is DRY and follows the team's coding standards.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] I have verified that my code does not introduce a debug warning in my local environment.
[x] I have verified that the functional tests work in my local environment.
[ ] I have verified that all automated tests pass or have provided a detailed comment about why I am submitting with a failed pipeline.
[ ] I have verified that any dependent changes have been merged and published in downstream commits.
[x] I have added a link to this Pull Request in the Asana task.
[x] I have moved the Asana task to "Ready for Functional Review".
[x] I have left a comment in Asana and GitHub tagging a team member with a request for `review.
Testing
Follow the steps below to test the changes in this PR.
Pull down branch to local site
Build local site successfully with no errors related to missing image
Functional tests
As the functional tester for this pull request, I verify that:
[ ] It [does something]
[ ] it does not...
[ ] As an admin, I can...
[ ] As a user, I can...
Once testing is complete, notify the author of any failed tests and move the task to "Kick back" in Asana. If all tests pass, move the task to "Ready for Code Review" in Asana and tag a team member for code review.
Code review
As the code reviewer for this pull request, I verify that:
[ ] All automated tests have passed.
[ ] The code is written (or documented) in a way that is easy to understand.
[ ] The code is free of obvious errors.
[ ] The code is free of obvious duplication.
[ ] The code follows our coding standards.
[ ] The code is sanitized or escaped appropriately for any SQL or XSS injection possibilities.
Once testing is complete, notify the author of any failed tests and move the task to "Kick back" in Asana or continue with the "merging" steps below.
Merging
As the individual merging this pull request, I verify that:
[ ] All automated tests have passed.
[ ] All functional tests have passed.
[ ] All code review tests have passed.
[ ] I have moved the task to "Ready to Deploy" in Asana and notified the pull request author.
Changes proposed in this pull request
This PR removes unused icons and pattern files.
Icons:
Patterns:
Closes https://app.asana.com/0/1203895219649773/1206815771033351/f https://app.asana.com/0/1203895219649773/1206815771033356/f
Pre-submit checklist
As the author of this pull request, I verify that:
master
.Testing
Follow the steps below to test the changes in this PR.
Functional tests
As the functional tester for this pull request, I verify that:
Once testing is complete, notify the author of any failed tests and move the task to "Kick back" in Asana. If all tests pass, move the task to "Ready for Code Review" in Asana and tag a team member for code review.
Code review
As the code reviewer for this pull request, I verify that:
Once testing is complete, notify the author of any failed tests and move the task to "Kick back" in Asana or continue with the "merging" steps below.
Merging
As the individual merging this pull request, I verify that: