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

Use either the id from input_html or rails default #58

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Use either the id from input_html or rails default #58

wants to merge 2 commits into from

Conversation

waldyr
Copy link

@waldyr waldyr commented Oct 26, 2017

Fixes #41. It was happening due to having at least two inputs with the same id making trix to always get the first one.

One work around is to set different html ids for each hidden input input_html but the trix wrapper won't set the input correctly. This PR fixes 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.

Not setting value when more than one trix editor on the page.
1 participant