ionic-framework icon indicating copy to clipboard operation
ionic-framework copied to clipboard

Remove ion-toolbar from being captured in screenshots where its appearance doesn't matter

Open gnbm opened this issue 10 months ago • 1 comments

Currently, any time ion-toolbar changes in appearance, a lot of screenshots have to be updated. Most of them are only being updated because they include ion-toolbar for labeling the test; the change in them doesn’t matter because the test isn’t meant to verify the look of the toolbar.

The following groups of tests should be updated so that the test does not include the toolbar, since it doesn’t make sense to test the toolbar as part of the screenshot test in them:

  • badge-basic
  • button
  • item
  • list-header
  • progress-bar
  • searchbar
  • toast
  • toggle

Some of these might have one or two tests where ion-toolbar is used in the component they’re testing, but most of them do not, and all of the components listed here have tests that need to be updated to be refocused so they don’t include the ion-toolbar in the screenshot capture.

gnbm avatar May 25 '25 12:05 gnbm

This issue has been labeled as help wanted. This label is added to issues that we believe would be good for contributors.

If you'd like to work on this issue, please comment here letting us know that you would like to submit a pull request for it. This helps us to keep track of the pull request and make sure there isn't duplicated effort.

For a guide on how to create a pull request and test this project locally to see your changes, see our contributing documentation.

Thank you!

ionitron-bot[bot] avatar May 25 '25 12:05 ionitron-bot[bot]