Skip to content

Conversation

shaswata-26
Copy link

@shaswata-26 shaswata-26 commented Sep 1, 2025

User description

This PR updates the Limitbox form field documentation to include the new showall option.

  1. Added explanation of the showall attribute (true to display "ALL" in the dropdown, false to hide it).
  2. Updated the options list in the docs.
  3. Helps clarify usage for developers using LimitboxField.php.

PR Type

Documentation


Description

  • Added documentation for showall option in Limitbox form field

  • Updated example with showall="true" attribute


Diagram Walkthrough

flowchart LR
  A["Limitbox Field"] --> B["showall option"]
  B --> C["Controls 'All' option visibility"]
  C --> D["Updated documentation"]
Loading

File Walkthrough

Relevant files
Documentation
limitbox.md
Document showall option for Limitbox field                             

docs/general-concepts/forms-fields/standard-fields/limitbox.md

  • Added showall option documentation with true/false values
  • Updated example code to include showall="true" attribute
  • Added explanation of "All" option visibility control
+5/-0     

Copy link
Contributor

qodo-merge-pro bot commented Sep 1, 2025

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Consistency

Verify that the default behavior described for showall matches the actual default in LimitboxField.php, including exact capitalization of the "All" label and localization.

- **showall** (optional) controls whether an "All" option is displayed in the dropdown list.
  - **true**: shows the "All" option
  - **false**: hides the "All" option (default)
Example Accuracy

Ensure the example attribute name and value type (showall="true") are correct per XML schema and that other examples in the docs are updated consistently.

           showall="true"
           onchange="this.form.submit();"
    />
</fields>

</details>

</td></tr>
</table>

Copy link
Contributor

qodo-merge-pro bot commented Sep 1, 2025

PR Code Suggestions ✨

No code suggestions found for the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant