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

Has visibility #229

Closed
wants to merge 2 commits into from
Closed

Has visibility #229

wants to merge 2 commits into from

Conversation

atmonshi
Copy link
Member

@atmonshi atmonshi commented Jan 8, 2024

No description provided.

Copy link

what-the-diff bot commented Jan 8, 2024

PR Summary

  • Introduction of a new parameter $hasVisibility
    A new parameter called $hasVisibility is added to the method appendFilamentComponentsOptions. This applies to add-fields.md, Bolt.php, CheckboxList.php, FileUpload.php, Radio.php, Select.php, and TextInput.php files. This tweak enhances the process by which the software interacts with certain components.

  • Integration of $hasVisibility in Bolt.php
    In Bolt.php, we've added a new variable, $hasVisibility, to help determine if there are fields with active visibility options in a given section. This helps in managing fields that are visible or hidden, thus enhancing the user interface experience for users.

  • Use of $hasVisibility in various files
    The $hasVisibility parameter is also used in CheckboxList.php, FileUpload.php, Radio.php, Select.php, and TextInput.php files. This modification strengthens how the program can handle various tasks associated with these components.

  • Typo fix in method calls in several files
    Finally, there are corrections made on a typographical error found in the method call to getFieldCollectionItemsList within multiple files. This correction ensures the smooth operation of various tasks.

@atmonshi atmonshi closed this Jan 8, 2024
@atmonshi atmonshi deleted the has-visibility branch January 8, 2024 22:47
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.

1 participant