Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Related issue on the OD project on GitHub: https://github.com/drnikki/open-demographics/issues/53
Proposed resolution
After much discussion in the #ddi-contrib-team channel on the Drupal Slack, we've decided to move forward with a single field for now.
We won't look into any performance or user experience improvements in this issue per comment #4.
Remaining tasks
- Create a follow-up issue for user experience improvement for checkboxes (applies to Drupal 8 as well?) using the issue template with details from #3 and #4.
Comment | File | Size | Author |
---|---|---|---|
#13 | Screen Shot 2019-02-17 at 2.50.31 PM.png | 37.08 KB | Darren Oh |
#11 | interdiff_9_11.patch | 8.09 KB | alexdmccabe |
#11 | gender-single_field-3015379-11-7.patch | 19.7 KB | alexdmccabe |
#9 | interdiff_4_9.txt | 8.68 KB | alexdmccabe |
#9 | gender-single_field-3015379-9-7.patch | 14.36 KB | alexdmccabe |
Comments
Comment #2
alexdmccabeComment #3
mradcliffeI tried out the patch on simplytest.me. Everything seems to be fairly functional.
Attached a sample screenshot of flexing the checkboxes element:
Comment #4
alexdmccabeComment #5
mradcliffeI've updated the issue summary based on discussions in Drupal Slack today listing the remaining tasks for the issue.
Comment #6
volkswagenchickComment #7
volkswagenchickComment #8
alexdmccabeI just noticed that we don't actually have a field formatter anymore. I'm going to work on that today.
Comment #9
alexdmccabeComment #10
alexdmccabeComment #11
alexdmccabeWorked on tests for the field at #fldc19.
Comment #12
alexdmccabeComment #13
Darren OhPatch 11 works.
Comment #14
Darren OhComment #15
Darren OhComment #16
Darren OhComment #18
alexdmccabeFixed! Thanks everyone!
Comment #19
alexdmccabeComment #20
alexdmccabeComment #21
alexdmccabeComment #22
alexdmccabe