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.

Sign up for GitHub

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

Jump to bottom

CommandButton: add inline Ajax status indicator #8301

Closed
jepsar opened this issue Jan 19, 2022 · 2 comments · Fixed by #8302
Closed

CommandButton: add inline Ajax status indicator #8301

jepsar opened this issue Jan 19, 2022 · 2 comments · Fixed by #8302
Assignees
@jepsar
Labels
new feature
Milestone
12.0.0

Comments

@jepsar
Copy link
Member

jepsar commented Jan 19, 2022 •
edited

The icon or text can be temporarily replaced with a spinner while an Ajax request is in progress.

commandbutton

Note that the speed of the GIF is slower than real time (don't want to spend time on getting the GIF right).

<h5>Ajax Disabling and Status Icon</h5>
<p:commandButton value="Disable" action="#{buttonBasicView.sleep()}" />
<p:commandButton value="Status" disableOnAjax="false" action="#{buttonBasicView.sleep()}" />
<p:commandButton value="Both + Icon" icon="pi pi-check" action="#{buttonBasicView.sleep()}" />
jepsar added a commit to jepsar/primefaces that referenced this issue Jan 19, 2022
@jepsar
Fix primefaces#8301: CommandButton: add inline Ajax status indicator
5eedd65
@jepsar jepsar mentioned this issue Jan 19, 2022
Merged
@Rapster
Copy link
Member

Rapster commented Jan 19, 2022

I like the idea. Do you have a gif to share to see what it look like?

@melloware melloware added this to the 12.0.0 milestone Jan 19, 2022
@klaros-testmanagement
Copy link

Great idea, I really like this too!

melloware pushed a commit that referenced this issue Apr 15, 2022
@jepsar
Fix #8301: CommandButton: add inline Ajax status indicator (#8302)
d6d48b8 
* Fix #8301: CommandButton: add inline Ajax status indicator

* Add property to JSdoc

* Really!?

* Add class and CSS

* Fix disable only

* Doc

* Fixed typo and add ui-state-loading to commandLink

* Add form

* disableOnAjax default true; remove inlineAjaxStatus

* Remove inlineAjaxStatus doc

* Add note on status indicator
@jacekzwroclawia jacekzwroclawia mentioned this issue Aug 5, 2022
Closed
jepsar added a commit to jepsar/primefaces that referenced this issue Aug 31, 2022
@jepsar
primefaces#8301 doc default
97e3be0
melloware pushed a commit that referenced this issue Aug 31, 2022
@jepsar
#8301 doc default (#9163)
0fa4e11
@jepsar jepsar mentioned this issue Oct 11, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jepsar jepsar

Labels
new feature
Projects
None yet
Milestone
12.0.0
Development

Successfully merging a pull request may close this issue.

Fix #8301: CommandButton: add inline Ajax status indicator jepsar/primefaces
4 participants
@melloware @jepsar @Rapster @klaros-testmanagement