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

This fixes the issue #136 for me #137

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

Conversation

owenfi
Copy link

@owenfi owenfi commented May 1, 2017

To recap the issue, if the formset starts hidden then the remove links don't appear.

I'm not sure what this might break, but seems to work so far for me...

To recap the issue, if the formset starts hidden then the remove links don't appear.

I'm not sure what this might break, but seems to work so far for me...
@ademus4
Copy link

ademus4 commented Jul 28, 2017

This was also an issue for me using Bootstrap tabs where I have a formset per tab. Current behavior isn't consistent with the add button since this appears on the hidden tab, but not the remove button. This change matches what I've had to do myself to get it to work.

@craiga
Copy link

craiga commented Nov 21, 2018

If the maintainer of this repo is still around, it would be great to get this merged.

I'm not sure what the purpose of not doing this for invisible forms might've been?

@elo80ka elo80ka self-assigned this Jun 20, 2019
@sebastianhafstrom
Copy link

Would be great to see this merged!

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.

None yet

5 participants