Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

Update WDIO snapshots #2081

Merged
merged 1 commit into from
Mar 14, 2024
Merged

Update WDIO snapshots #2081

merged 1 commit into from
Mar 14, 2024

Conversation

saket2403
Copy link
Contributor

Summary

What was changed:
Fixed wdio snapshots failing on main

Why it was changed:

Testing

This change was tested using:

  • WDIO
  • Jest
  • Visual testing (please attach a screenshot or recording)
  • Other (please describe below)
  • No tests are needed

Reviews

In addition to engineering reviews, this PR needs:

  • UX review
  • Accessibility review
  • Functional review

Additional Details

This PR resolves:

UXPLATFORM-XXXX


Thank you for contributing to Terra.
@cerner/terra

@saket2403 saket2403 self-assigned this Mar 14, 2024
@github-actions github-actions bot temporarily deployed to preview-pr-2081 March 14, 2024 09:40 Destroyed
@saket2403 saket2403 changed the title Fix Wdio snapshots failing main build Update WDIO snapshots Mar 14, 2024
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the old blue focus used instead of dashed border ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks to me like it's using the default focus style from terra-form-input for some reason. I'm not sure why since there haven't been any changes to terra-form-input for quite some time.

https://github.com/cerner/terra-core/blob/main/packages/terra-form-input/src/Input.module.scss#L51

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This might be the reason WDIO applies wrong border: https://github.com/cerner/terra-framework/pull/2064/files

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But it is only mozilla specific styling and not applied to date input

@sdadn
Copy link
Contributor

sdadn commented Mar 14, 2024

Looks like the new screenshots are correct; it is cause by a styling defect when shifting focus from date-input -> date-input-field -> date-input and it was reproducible on multiple devices. We should merge this change for now since screenshots should represent the "current" state of code and correct the defect at a later date.

@saket2403 saket2403 merged commit 78a1c6b into main Mar 14, 2024
22 checks passed
@saket2403 saket2403 deleted the fix-main branch March 14, 2024 17:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants