Skip to content

feat(payment): Get applied styles for HTML pseudo elements#2381

Merged
PavlenkoM merged 1 commit intobigcommerce:masterfrom
PavlenkoM:feat/get-applied-styles-for-pseudo-elements
Jun 20, 2025
Merged

feat(payment): Get applied styles for HTML pseudo elements#2381
PavlenkoM merged 1 commit intobigcommerce:masterfrom
PavlenkoM:feat/get-applied-styles-for-pseudo-elements

Conversation

@PavlenkoM
Copy link
Contributor

@PavlenkoM PavlenkoM commented Jun 19, 2025

What?

Modify getAppliedStyles from dom-utils package

Why?

To use functionality of window.getComputedStyle.
And get ability to get styles for HTML pseudo-element.

Testing / Proof

Manually checked

@bcsnyk
Copy link
Contributor

bcsnyk commented Jun 19, 2025

🎉 Snyk checks have passed. No issues have been found so far.

code/snyk check is complete. No issues have been found. (View Details)

@PavlenkoM PavlenkoM force-pushed the feat/get-applied-styles-for-pseudo-elements branch from f0bef48 to 5042651 Compare June 19, 2025 13:00
@PavlenkoM PavlenkoM force-pushed the feat/get-applied-styles-for-pseudo-elements branch from 5042651 to 171bed8 Compare June 19, 2025 13:07
@PavlenkoM PavlenkoM marked this pull request as ready for review June 19, 2025 13:10
@PavlenkoM PavlenkoM requested a review from a team as a code owner June 19, 2025 13:10
@PavlenkoM PavlenkoM merged commit bed7564 into bigcommerce:master Jun 20, 2025
8 of 9 checks passed
@bc-launchbay
Copy link

checkout-js deployed to Staging US
success 20250620072951 by Launchbay

@github-actions
Copy link

⚡️ Lighthouse results

🖥️ Desktop:

Category Score
🟠 Performance 77
🟢 Accessibility 91
🟠 Best practices 78

📱 Mobile:

Category Score
🟠 Performance 52
🟢 Accessibility 94
🟠 Best practices 78

@bc-launchbay
Copy link

checkout-js deployed to Production US
success 20250620072951 by Launchbay

@bc-launchbay
Copy link

checkout-js deployed to Integration US
success 20250620072951 promoted by https://launchbay.bigcommerce.net/projects/535/releases/258028 being deployed. Deployed by Launchbay

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.

7 participants