Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test(interstitialscreen): Add default state AVT check #5170

Conversation

sangeethababu9223
Copy link
Member

Closes #5063

Add default state AVT check for InterstitialScreen
Since there is no default option in the story, added AVT check for Full Screen.

What did you change?

Created e2e/components/InterstitialScreen/InterstitialScreen-test.avt.e2e.js
Had to disable lint for non interactive tabindex since the div is scrollable.

How did you test and verify your work?

yarn avt

@sangeethababu9223 sangeethababu9223 requested a review from a team as a code owner May 14, 2024 07:41
@sangeethababu9223 sangeethababu9223 requested review from kennylam and ariellalgilmore and removed request for a team May 14, 2024 07:41
Copy link

netlify bot commented May 14, 2024

Deploy Preview for carbon-for-ibm-products ready!

Name Link
🔨 Latest commit dce3581
🔍 Latest deploy log https://app.netlify.com/sites/carbon-for-ibm-products/deploys/6643159eb354650008adf7f7
😎 Deploy Preview https://deploy-preview-5170--carbon-for-ibm-products.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented May 14, 2024

Deploy Preview for carbon-for-ibm-products ready!

Name Link
🔨 Latest commit 6f1733e
🔍 Latest deploy log https://app.netlify.com/sites/carbon-for-ibm-products/deploys/6644440e9493c500084363b4
😎 Deploy Preview https://deploy-preview-5170--carbon-for-ibm-products.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@szinta szinta self-requested a review May 15, 2024 04:07
cspell.json Outdated Show resolved Hide resolved
Copy link
Contributor

@szinta szinta left a comment

Choose a reason for hiding this comment

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

Looks good. One small change is required as per #5202

@sangeethababu9223 sangeethababu9223 added this pull request to the merge queue May 21, 2024
Merged via the queue into carbon-design-system:main with commit ba65093 May 21, 2024
16 checks passed
@sangeethababu9223 sangeethababu9223 deleted the test/interstitialscreen-avt branch May 21, 2024 19:38
paul-balchin-ibm pushed a commit to paul-balchin-ibm/ibm-products that referenced this pull request May 23, 2024
…system#5170)

* test(InterstitialScreen): default avt

* test(InterstitialScreen): add default avt

* test(InterstitialScreen): disable lint for tabindex in scollable div

* test(InterstitialScreen): add default avt

* test(interstitialscreen): remove interstitialscreen from cspell.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add default state AVT check for InterstitialScreen
3 participants