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

New: Added button render position config (fixes #12) #23

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

guywillis
Copy link
Contributor

Fixes: #12

New

  • Added new config option _buttonPosition
  • _buttonPosition can be changed to left, right, top, bottom
  • Changes the button render position in relation to the instruction text depending on value selected

@guywillis guywillis requested a review from swashbuck January 17, 2025 15:59
@guywillis guywillis self-assigned this Jan 17, 2025
@@ -36,6 +36,19 @@
"validators": [],
"translatable": true
},
"_buttonPosition": {
"type": "string",
"required": true,
Copy link
Contributor

@kirsty-hames kirsty-hames Feb 10, 2025

Choose a reason for hiding this comment

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

Should we set required to false seeing as we have a default value? Same goes for other model config.

Copy link
Contributor

@kirsty-hames kirsty-hames left a comment

Choose a reason for hiding this comment

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

Works as expected thanks but I've added a query regarding the schema requirement.

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.

Add options to configure button position
2 participants