-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
Improve payment form #61
Comments
Hi @facundofarias, We would gladly appreciate any improvement to the code. Please note that we have a strict Content Security Policy ( https://developer.mozilla.org/en-US/docs/Web/Security/CSP/Using_Content_Security_Policy ) enabled on the pages, so it's possible that the various javascript plugins will not work. Additionally we want to support mobile browsers, so the widget must be mobile aware. The second linked plugin seems better, as we would like to keep the various UI elements as simple as possible, without surprising elements. Thank you :D |
Yes, of course. The idea in here is to discuss about it, and if he have an agreement, then we can help with the feature development. I haven't been digging too much into the code, so this will help me understand a bit more about the product itself. |
I agree with @syjer: the second plugin would fit better with our UI (which is intended to be minimalistic btw). |
Hi @facundofarias, do you have any news? thank you very much! |
closed for inactivity |
I was thinking that maybe we can use something like:
I can work on the integration if you think it might be Okay.
The text was updated successfully, but these errors were encountered: