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

Decide whether and how to maintain the CfAFormBuilder #124

Closed
hartsick opened this issue Sep 23, 2019 · 3 comments
Closed

Decide whether and how to maintain the CfAFormBuilder #124

hartsick opened this issue Sep 23, 2019 · 3 comments
Labels
discussion Subjects for group discussion

Comments

@hartsick
Copy link
Contributor

hartsick commented Sep 23, 2019

The CfAFormBuilder is not used by any active projects, and Norris had raised some concerns about the form styling with this design update: #123

Should we continue to maintain the styleguide's formbuilder? And if so, should we re-build it from GCF's form builder or keep what we currently have?

@bengolder as someone who might be using this soon, I'd love your thoughts on the question above!

@hartsick hartsick added the discussion Subjects for group discussion label Sep 23, 2019
@bengolder
Copy link
Contributor

I think it's useful to have a formbuilder. I think there are some decisions & improvements to make to keep it useful. Assuming I have a 2nd engineer on my team, we may be able to contribute back to it. In my usage during the prototype I frequently created custom formbuilder methods because the current formbuilder didn't quite meet my needs (but it's close). In particular, I wanted to be able to create <h1> labels for single question pages for better screen reader hierarchy.

https://github.com/codeforamerica/eitc/blob/master/app/helpers/eitc/eitc_form_builder.rb

In terms of choosing between GCF & the current formbuilder, I would lean towards whichever feels most up-to-date with our accessibility standards & design and is most widely used. I'm guessing that would be GCF, but I'm not sure.

Since GCF has its own form builder, we could use EITC as an initial use case for a shared formbuilder, and review the work with other styleguide stakeholders, and fold it back in to assist later prototyping efforts.

@hartsick
Copy link
Contributor Author

hartsick commented Nov 7, 2019

Notes from discussion about FormBuilder here: https://docs.google.com/document/d/1LfwFEr1VX0uTTpBQdhjSrdM_ycAsJkiAG_FLzu3PHko/edit

@hartsick
Copy link
Contributor Author

hartsick commented Apr 1, 2021

Closed by #204

@hartsick hartsick closed this as completed Apr 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Subjects for group discussion
Projects
None yet
Development

No branches or pull requests

2 participants