Skip to content

Forms #69

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

Merged
merged 16 commits into from
Jul 26, 2024
Merged

Forms #69

merged 16 commits into from
Jul 26, 2024

Conversation

pinkeshmars
Copy link
Collaborator

Description

Provide a brief overview of what this documentation update is about. Explain what sections or topics are being added or revised.

Linear ticket and magic word Fixes DEVR-385

Type of change

  • Typo fix
  • New feature
  • Enhancement to current docs
  • Removed outdated references
  • Update assets

@pinkeshmars pinkeshmars requested a review from PoojaB26 July 25, 2024 05:48
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@PoojaB26
Copy link
Collaborator

hey @pinkeshmars I thought we were using Arcade for uploading GIFs, and not storing locally. This will affect the performance of loading the site. Some of them are 6MB+, in any way, any image that is above 1MB should definitely be removed or optimized further. Can you move them to Arcade? Atleast the heavy files?

Copy link
Collaborator

@PoojaB26 PoojaB26 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added some comments and since we are close to release, lets fix the links too. Also add custom slug paths to the non component files like forms(built in widgets and components currently will follow the hierarchy as it is, we will fix it later after all components are merged).

Also add SEO keywords and tags, chatGPT can help with that if you add the current front matter and first paragraph, and ask GPT to output modified front matter with additional keywords and tags in front matter.

@@ -0,0 +1,906 @@
---
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename the file to all small, it is currently in PascalCase.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or text-field.md

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!


# Overview
Forms are a crucial component of any application, providing a structured way to gather user input. In FlutterFlow, creating forms is straightforward and highly customizable, allowing you to efficiently collect various types of data. FlutterFlow offers a variety of input fields for gathering different types of data, making it versatile for creating forms in your applications.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a :::tip and add links to relevant docs here

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also maybe we dont need a new page for this? like this file can be renamed as forms.md and be opened when you click on forms dropdown?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!


### Validating input

You can validate the *TextField* value by wrapping it inside the [Form](#) widget and adding the validation criteria.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this, we have Form highlighted as a hyperlink, but we dont have Form as a separate file in this PR, right? It should be part of Forms PR right? If yes, lets add that and also fix the link here since we are close to launch.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see we can link it to Form Validation page.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added!

You can validate the *TextField* value by wrapping it inside the [Form](#) widget and adding the validation criteria.

:::info
**Tip**: Filtering ensures that only the allowed characters or values are entered, whereas the [validation](#) checks the entire input data against certain criteria. Both techniques can be used together or independently to ensure the correctness of user input in a TextField widget.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can directly use tip admonition.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated!

| --- | --- |
| Credit card number (e.g., 3424 4353 5453 3535) | #### #### #### #### |
| Custom date (e.g., 12-Jan-2023) | ##-AAA-#### |

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add p tag here.

Copy link
Collaborator Author

@pinkeshmars pinkeshmars Jul 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

p tag didn't have any effect here so added a
tag.

@@ -0,0 +1,906 @@
---
slug: textfield
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

slug: resources/forms/textfield

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@@ -0,0 +1,97 @@
---
slug: checkboxgroup
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

follow slug rule like textfield

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@@ -0,0 +1,310 @@
---
slug: dropdown
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

follow slug rule like textfield

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@@ -0,0 +1,178 @@
---
slug: radiobutton
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

follow slug rule like textfield

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@@ -1,5 +1,10 @@
---
slug: overview
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

slug: resources/forms

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done!

@PoojaB26 PoojaB26 merged commit 51eb8dd into main Jul 26, 2024
@PoojaB26 PoojaB26 deleted the pinkesh/forms branch July 26, 2024 09:10
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