stripe / stripe-js

Loading wrapper for Stripe.js
https://stripe.com/docs/js
MIT License
622 stars 153 forks source link

[BUG]: Payment element - Placeholder color setting not respected - it's getting overridden with the default #607

Closed MikesGlitch closed 4 months ago

MikesGlitch commented 4 months ago

What happened?

I've set the following property in the appearance option object

colorTextPlaceholder

It appears on the screen at first, then it gets overridden by the default colour (presumably as it loads in the css).

I have a repro here: https://codesandbox.io/p/sandbox/stripe-ghmm9r

Placeholder text should be red, but it always gets put back to the default gray.

Environment

No response

Reproduction

https://codesandbox.io/p/sandbox/stripe-ghmm9r

tylersmith-stripe commented 4 months ago

Hey @MikesGlitch , your codesandbox is throwing an IntegrationError due to invalid clientSecret formatting.

I created a fork of your repro that builds successfully, and things are working "as expected" from my end. The country select is not red because the value is filled, and not displaying a placeholder. Feel free to re-open if you can narrow the bug down.

image

MikesGlitch commented 4 months ago

Bizzare, I get this on edge: image

It works on firefox though!
image

Yeah I deliberately removed the secret.

Can you confirm it works/doesn't work for you on edge? Our users are using all kinds of browsers - it was reported on Edge though

tylersmith-stripe commented 4 months ago

I'll take a look at Edge!

tylersmith-stripe commented 4 months ago

@MikesGlitch What version of Edge are you using? It looks correct to me in Edge 109. image

MikesGlitch commented 4 months ago

Sorry, I've just discovered this extension is to blame: ProWritingAid: Grammar Checker & Paraphrasing Tool

It must have been setting something on the fields. When I turn it off, it works as expected. This is fine to close