Skip to content

Conversation

mapsandapps
Copy link
Contributor

@mapsandapps mapsandapps commented Nov 2, 2023

Issue number: N/A


What is the current behavior?

Sometimes the screenshot gets captured as
actual

when it should be
expected

What is the new behavior?

  • Hopefully the screenshot will be more consistent.

Does this introduce a breaking change?

  • Yes
  • No

Other information

@github-actions github-actions bot added the package: core @ionic/core package label Nov 2, 2023
@mapsandapps mapsandapps marked this pull request as ready for review November 2, 2023 17:38
@mapsandapps mapsandapps added this pull request to the merge queue Nov 2, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 2, 2023
@liamdebeasi liamdebeasi added this pull request to the merge queue Nov 2, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 2, 2023
@liamdebeasi liamdebeasi added this pull request to the merge queue Nov 2, 2023
Merged via the queue into main with commit 2748851 Nov 2, 2023
@liamdebeasi liamdebeasi deleted the flaky-picker-test branch November 2, 2023 19:17
liamdebeasi pushed a commit that referenced this pull request Nov 2, 2023
… test (#28456)

Issue number: N/A

---------

<!-- Please do not submit updates to dependencies unless it fixes an
issue. -->

<!-- Please try to limit your pull request to one type (bugfix, feature,
etc). Submit multiple pull requests if needed. -->

<!-- Please describe the current behavior that you are modifying. -->

Sometimes the screenshot gets captured as

![actual](https://github.com/ionic-team/ionic-framework/assets/14926794/dd596788-f871-45b8-ad0b-f8d3682404e4)

when it should be

![expected](https://github.com/ionic-team/ionic-framework/assets/14926794/4807e447-3a02-44e5-95a3-04f1ee742072)

<!-- Please describe the behavior or changes that are being added by
this PR. -->

- Hopefully the screenshot will be more consistent.

- [ ] Yes
- [x] No

<!-- If this introduces a breaking change, please describe the impact
and migration path for existing applications below. -->

<!-- Any other information that is important to this PR such as
screenshots of how the component looks before and after the change. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core @ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants