-
Notifications
You must be signed in to change notification settings - Fork 20
Forms review #700
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
Forms review #700
Conversation
tomasMizera
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing improvement! Thank you! I suggest we run internal feedback loop now, collect comments from others and then incorporate that with my comments in one go.
| { text: 'How to Use Hyperlinks', link: '/layer/external-link/' }, | ||
| { text: 'How to Open a File', link: '/layer/open-file/' }, | ||
| { text: 'How to Use Hyperlinks', link: '/layer/external-link/' }, | ||
| { text: 'How to Open a Link to a Navigation App', link: '/layer/link-to-navigation/' }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about making this shorter? Something like
| { text: 'How to Open a Link to a Navigation App', link: '/layer/link-to-navigation/' }, | |
| { text: 'How to Open a Navigation App', link: '/layer/link-to-navigation/' }, |
| { text: 'QR Code', link: '/layer/qr-code/' }, | ||
| { text: 'Selecting a Single or Multiple Values', link: '/layer/value-select/' }, | ||
| { text: 'Selecting Single or Multiple Values', link: '/layer/value-select/' }, | ||
| { text: 'Photos', link: '/layer/photos/' }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we wanted to put photos higher in the list and relations lower
|
|
||
| Therefore, **we strongly recommend creating a new field for primary keys and using UUIDs generated by the `uuid()` function when a feature is created [as described below](#generating-unique-ids-uuid)**. UUID (Universally Unique Identifier) ensures that the primary keys are unique even after synchronisation and that the correct features are linked. | ||
|
|
||
| ## One to many relation setup overview |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about
| ## One to many relation setup overview | |
| ## Setup overview |
| 1. [Create unique UUID field](#generating-unique-ids-uuid) for your referenced (parent) layer and a field for storing foreign keys in the referencing (child) layer | ||
| 3. Define the relation in [Project Properties](#setting-up-relation-in-qgis) | ||
| 4. [Configure Attributes forms](#attributes-forms-configuration-for-1-n-relations) of both layers as needed | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd add a sentence that explains that we are going to talk about these now
| Let's walk through each of the points individually now. |
| :::tip Example projects available | ||
| To see the basic [attachment widget](#photo-attachment-widget-in-qgis) setup for photos, clone <MerginMapsProject id="documentation/form_setup" />. | ||
|
|
||
| If you want to [attach multiple photos to one feature](#how-to-attach-multiple-photos-to-one-feature) explore the public project <MerginMapsProject id="documentation/forms_multiple_photos" />. | ||
| ::: | ||
|
|
||
| ## Photo attachment widget in QGIS | ||
|
|
||
| :::tip Example project available | ||
| Clone <MerginMapsProject id="documentation/form_setup" /> to follow this example! | ||
| ::: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It feels too much to have two tips for public projects this close. Can we choose only one of them?
|
|
||
|  | ||
|
|
||
| ## Gallery? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about
| ## Gallery? | |
| ## Troubleshooting |
instead?
It could also include the common issue "why do I not see my relations in the form?" -> in drag&drop designer you need to manually move the relation into the form.
|
|
||
| Selecting values from a drop-down menu is faster than typing them in manually and it also ensures that there are no typos or spelling variations. Additionally, it is possible to create [drill-down (cascade) forms](#drill-down-forms), where a value selected in one field can affect the options available in another field. | ||
|
|
||
| |<div style="width:150px"> Widget </div> |Preview in the <MobileAppNameShort />| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the future I can imagine some GIFs here to showcase the drill-down functionality 🤔
| { text: 'Photos', link: '/layer/photos/' }, | ||
| { text: 'Informational Widgets', link: '/layer/info-widgets/' }, | ||
| ] }, | ||
| { text: 'Default Values', link: '/layer/default-values/', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am bit afraid this rich article will be skipped by people clicking on the arrow icon to expand the menu instead of clicking on the entry. Let's analyse that afterwards.
Redirects fail as src/field/photo-sketching/index.md was added to the main in the meantime.
Broken links fail due to 403 wiki.