Skip to content

Commit

Permalink
Merge branch 'main' into feature/134/autosuggest-testing-refactor
Browse files Browse the repository at this point in the history
  • Loading branch information
precious-onyenaucheya-ons authored Sep 17, 2024
2 parents 04d324e + 1e4c58d commit 740abf2
Show file tree
Hide file tree
Showing 7 changed files with 16 additions and 8 deletions.
4 changes: 4 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ backstop_data/bitmaps_test/
backstop_data/html_report/
backstop.config-for-docker.json

# Generated from lighthouse tests
.lighthouseci/
lighthouse/urls.json

# npm package folders
/components/
/page-templates/
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ It is sometimes useful to adjust the following settings when writing tests or di

## Testing - Visual regression tests

This project uses [Backstop JS](https://github.com/garris/BackstopJS) for visual regression testing. The tests run in Chrome headless using Pupeteer inside docker and run in three viewports; 1920 (desktop), 768 (tablet) and 375 (mobile). Reference images are stored in Git LFS and any approved changes will automatically be stored in Git LFS when pushed to the repository.
This project uses [Backstop JS](https://github.com/garris/BackstopJS) for visual regression testing. The tests run in Chrome headless using Puppeteer inside docker and run in three viewports; 1920 (desktop), 768 (tablet) and 375 (mobile). Reference images are stored in Git LFS and any approved changes will automatically be stored in Git LFS when pushed to the repository.

The visual tests will run automatically on pull requests and the result will be available in the Github Action logs. If the tests fail, the process for viewing the failures and approving changes will need to be handled locally using the following workflow and commands.

Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
2 changes: 2 additions & 0 deletions src/components/list/_list.scss
Original file line number Diff line number Diff line change
Expand Up @@ -109,8 +109,10 @@
}
.ons-list__prefix,
.ons-list__suffix {
@extend .ons-u-m-no;
&--icon-check .ons-icon {
fill: var(--ons-color-leaf-green) !important;
@extend .ons-u-mr-s;
}
}
}
Expand Down
4 changes: 3 additions & 1 deletion src/components/list/example-summary-list.njk
Original file line number Diff line number Diff line change
@@ -1,10 +1,12 @@
{% from "components/list/_macro.njk" import onsList %}
{{
onsList({
"iconPosition": "before",
"variants": "summary",
"itemsList": [
{
"text": 'birth certificate'
"text": 'birth certificate',
"iconType": "check"
},
{
"text": 'national insurance number'
Expand Down

0 comments on commit 740abf2

Please sign in to comment.