Skip to content

Checkboxes fix - #472

Open
NicoHood wants to merge 2 commits into
getgrav:developfrom
NicoHood:checkboxes_fix
Open

Checkboxes fix#472
NicoHood wants to merge 2 commits into
getgrav:developfrom
NicoHood:checkboxes_fix

Conversation

@NicoHood

@NicoHood NicoHood commented Jan 9, 2021

Copy link
Copy Markdown
Contributor

Comment thread templates/forms/default/data.html.twig
Comment thread classes/Form.php
Comment thread templates/forms/fields/checkboxes/checkboxes.html.twig Outdated
Comment thread templates/forms/default/data.html.twig
@NicoHood

NicoHood commented Feb 1, 2021

Copy link
Copy Markdown
Contributor Author

Would you mind merging this?

@rhukster

rhukster commented Feb 1, 2021

Copy link
Copy Markdown
Member

I will look at this one again now I've released Form.

@NicoHood

Copy link
Copy Markdown
Contributor Author

I have rebased my changes, simply removing the translation fix, as it was merged in #494

@mahagr

mahagr commented Feb 11, 2021

Copy link
Copy Markdown
Member

I won't accept this fix as it won't work in admin (it doesn't use form plugin, just the field), but I have an idea how to fix this.

@mahagr mahagr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I will try another approach.

@NicoHood

Copy link
Copy Markdown
Contributor Author

Any progress on this issue?

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.

Multiple issues with checkboxes

3 participants