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

[INFO] Is it possible to use only icons in ActionDialogue? #4586

Closed
kolliLakshmaiah1219 opened this issue Sep 4, 2024 · 5 comments · Fixed by #4593
Closed

[INFO] Is it possible to use only icons in ActionDialogue? #4586

kolliLakshmaiah1219 opened this issue Sep 4, 2024 · 5 comments · Fixed by #4593

Comments

@kolliLakshmaiah1219
Copy link

Is it possible to use only icons in ActionDialogue?

@kolliLakshmaiah1219 kolliLakshmaiah1219 changed the title [INFO] [INFO] Is it possible to use only icons in ActionDialogue? Sep 4, 2024
@sbwalker
Copy link
Member

sbwalker commented Sep 4, 2024

@kolliLakshmaiah1219 there is a parameter of IconOnly ie. <ActionDialog IconOnly="true" ...

@kLakshmaiah
Copy link

<ActionDialog Header="Delete Survey"
Message="@("Are You Sure You Wish To Delete The Survey?")"
IconName="@Icons.Trash"
IconOnly="true"
Action="Delete" Security="SecurityAccessLevel.Edit"
Class="btn btn-danger"
OnClick="@(async () => await DeleteFaq())" /> this is my code but in UI look like this
ActionDialogImage

@sbwalker
Copy link
Member

sbwalker commented Sep 6, 2024

@mdmontesinos you added the enhancement for IconOnly in #4219 - can you please review this issue to determine if the component is working as expected or not?

@kolliLakshmaiah1219
Copy link
Author

how do i create stored procedures in oqtane ,can you provide any links for that

@mdmontesinos
Copy link
Contributor

@sbwalker It seems that my commit (#4219) broke localization and you fixed it back in (3648f99). However, this broke the IconOnly implementation and I did not realize, as you set the _openText variable after it should have been cleared if IconOnly was true.

I just submitted a PR (#4593) that handles both the localization and IconOnly.

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 a pull request may close this issue.

4 participants