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

Update button.handlebars adding after-icon #348

Merged
merged 1 commit into from
May 29, 2024

Conversation

amrutadotorg
Copy link
Contributor

{{#if after_icon}}: This new block checks if the after_icon parameter is provided. If it is, it renders the icon after the text. In this way, the template will support both an icon before and after the text, depending on which parameters (icon or after_icon) are provided.

Explanation
{{#if after_icon}}: This new block checks if the after_icon parameter is provided. If it is, it renders the icon after the text.
In this way, the template will support both an icon before and after the text, depending on which parameters (icon or after_icon) are provided.
@lovasoa
Copy link
Collaborator

lovasoa commented May 29, 2024

Great, thanks for the pr ! Can you also, in the same pull request, add the new item to the documentation: https://github.com/lovasoa/SQLpage/blob/main/examples/official-site/sqlpage/migrations/18_button.sql ?

Don't close this PR, just amend it.

@lovasoa lovasoa merged commit 8551e59 into sqlpage:main May 29, 2024
9 checks passed
lovasoa added a commit that referenced this pull request May 29, 2024
@lovasoa
Copy link
Collaborator

lovasoa commented May 29, 2024

@amrutadotorg : don't worry, I did it myself in e5a9649 . Thanks for your contribution !

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