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

Dynamic fields settings page #87

Closed
SayChi opened this issue Dec 6, 2019 · 1 comment
Closed

Dynamic fields settings page #87

SayChi opened this issue Dec 6, 2019 · 1 comment

Comments

@SayChi
Copy link

SayChi commented Dec 6, 2019

When toggling "Send confirmation email" or "Overwrite the e-mail template" in the settings page the corresponding fields hide/show, which is great. But if these switches are already off when the settings page is loaded the corresponding fields show by default (which they shouldn't, and toggling the switches off and on 'fixes' it).

I realise this is a small nitpicking issue, but it would be nice if those states were checked in the init.

@DynamiteGoesBoom
Copy link
Contributor

DynamiteGoesBoom commented Oct 23, 2020

@SayChi good catch. It looks like the Lightswitch states went awry.

Thanks for the heads up!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants