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

ACF 5.3.1 support #38

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

ACF 5.3.1 support #38

wants to merge 1 commit into from

Conversation

radist2s
Copy link

I implemented full support for ACF 5.3.1, and it should work for previous ACF versions.

@mrwweb
Copy link
Owner

mrwweb commented Oct 22, 2015

@radist2s Thank you! I'm testing this just now and it's mostly working. So far I've found a few issues:

  1. No single-row collapse for block layout
  2. Single row collapse doesn't work in row layout repeater nested in block layout
  3. No "collapse all rows" for flexible fields

This does seem close, so I'd love to put out your changes if you can fix these issues.

I'm hoping for a bigger rewrite sometime in the next year to improve the nested-repeater collapsing, but for now, this fix is greatly appreciated. Thank you!

@jesserosenfield
Copy link

jesserosenfield commented Sep 6, 2016

Just made some basic fixes for ACF 5.4.4 by updating class names in the CSS and JS files
https://github.com/jesserosenfield/ACF-Repeater-Collapser

Thank you @mrwweb !!!

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.

3 participants