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

James/en/992 #1467

Merged
merged 5 commits into from May 24, 2024
Merged

James/en/992 #1467

merged 5 commits into from May 24, 2024

Conversation

James-Baloyi
Copy link
Contributor

@James-Baloyi James-Baloyi commented May 8, 2024

When utilizing the Modal dialog box Entity Reference type, the modal buttons persistently display even when the Show Modal Buttons option is set to false. Similarly, the modal buttons continue to appear when Allow New Record is enabled on the Entity picker.

Steps to reproduce the behavior:
Navigate to 'https://function-adminportal-test.shesha.dev/dynamic/Boxfusion.SheshaFunctionalTests.Common/book-twenty-two'
Click on 'Add'
Open form designer mode.
Drag and drop an Entity Picker component and configure it.
Enable the Allow New Record option.
Ensure that Show Modal Buttons is set to false.
Save and render the form.
Click on the Entity Picker and select the Add New button.
Observe that the modal buttons persist despite the Show Modal Buttons setting.
Similarly, within the details form, add a new Entity Reference and select the modal box dialog entity type.
Do not select the Show Modal Buttons option.
Save and render the form.
Observe that the modal buttons still display.
Expected behavior
When Show Modal Buttons are set to false or when Allow New Record is enabled without the modal buttons option selected, the modal buttons should not appear in the Entity Picker or Entity Reference modal dialog.

image

@Lukeybooi Lukeybooi merged commit 4acfd07 into shesha-io:main May 24, 2024
1 of 2 checks passed
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.

None yet

2 participants