-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Grouped fields #8104
Comments
@quincylvania Hey, I want to start contributing to this project. Could you guide me a bit more on this issue. Would love to start off somewhere. |
Welcome, @karmanya007. This issue is actually a bit advanced, and there aren't that many introductory issues open right now. But if you're feeling intrepid then can read our contributing guide and jump in anywhere. |
We already have such fields - like operator and operator:type for schools. Can't we use the same for the ones above? If a user fills in the first field, other appear below it. |
@maro-21 Yes, these already work such that one field's appearance depends on a related field's value, but they should also be grouped together visually. |
@quincylvania I think, now I can take this one. Could you tell me how the grouping happens? |
iD has a few special fields that group related tags together, like Name, Address, Structure, Allowed Access, and Bike Lanes. Grouping is a nice way to make the inspector simpler and more intuitive. There are a lot more tags that could benefit from grouping, for example:
Instead of coding a special field type for every grouping, we should look into a reusable grouped field type that can handle simple combinations.
The text was updated successfully, but these errors were encountered: