Skip to content

Fixed label for bug with specified id - #213#221

Closed
datWav wants to merge 1 commit intobootstrap-ruby:masterfrom
datWav:label_fix
Closed

Fixed label for bug with specified id - #213#221
datWav wants to merge 1 commit intobootstrap-ruby:masterfrom
datWav:label_fix

Conversation

@datWav
Copy link
Copy Markdown

@datWav datWav commented Apr 24, 2015

No description provided.

lcreid pushed a commit to lcreid/rails-bootstrap-forms that referenced this pull request Jun 4, 2018
If provided, use option `id` to specify `for` attribute on label
for (text, email...) fields, checkbox, radio and select.
Solves bootstrap-ruby#342 issue and supercede bootstrap-ruby#221 and bootstrap-ruby#343 pull requests.

Partially solves bootstrap-ruby#251 (only the part regarding the id).
@lcreid
Copy link
Copy Markdown
Contributor

lcreid commented Oct 28, 2018

Thank you for your interest in improving bootstrap_form. bootstrap_form has been substantially changed since this PR was submitted. The upgrade to Bootstrap 4 introduced many changes that may have addressed this PR, or made it not applicable. Therefore, we're closing this PR.

If you still believe this PR is needed, please re-open it and update it to be mergeable with the current master. Our apologies for not responding sooner.

@lcreid lcreid closed this Oct 28, 2018
lcreid added a commit that referenced this pull request Mar 1, 2019
* Use option[:id] for label's *for* attribute

If provided, use option `id` to specify `for` attribute on label
for (text, email...) fields, checkbox, radio and select.
Solves #342 issue and supercede #221 and #343 pull requests.

Partially solves #251 (only the part regarding the id).

* Move require mocha/minitest as needed by newer version.

* Fix version of sqlite3 < 1.4 so tests will work.
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.

2 participants