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

[Feat]: Support 'pending' button states #3162

Closed
1 task done
benjamind opened this issue Apr 28, 2023 · 3 comments · Fixed by #3163
Closed
1 task done

[Feat]: Support 'pending' button states #3162

benjamind opened this issue Apr 28, 2023 · 3 comments · Fixed by #3163
Assignees
Labels
Component: Button enhancement New feature or request help wanted Extra attention is needed Spectrum Engineering

Comments

@benjamind
Copy link
Contributor

Code of conduct

  • I agree to follow this project's code of conduct.

Impacted component(s)

button

Description of the requested feature

We should implement the 'pending' button state as described here: https://spectrum.adobe.com/page/button/#Pending

Mockups or screenshots

No response

Implementation notes or ideas

No response

@benjamind benjamind added enhancement New feature or request triage An issue needing triage labels Apr 28, 2023
@najikahalsema najikahalsema added help wanted Extra attention is needed Component: Button Spectrum Engineering and removed triage An issue needing triage labels Oct 5, 2023
@Rajdeepc
Copy link
Contributor

Rajdeepc commented Oct 9, 2023

WIP PR: #3163

@Rajdeepc
Copy link
Contributor

Rajdeepc commented Nov 29, 2023

@benjamind I see a PR opened by you on this. Would you like us to take this forward from you or do you have some plans to land this. Let me know what works best for you so that we can add this to our upcoming sprint.

@benjamind
Copy link
Contributor Author

By all means please take this forward, I don't think I'll have time to land it any time soon! Thanks!

@Rajdeepc Rajdeepc assigned TarunAdobe and unassigned Rajdeepc Dec 1, 2023
Westbrook pushed a commit that referenced this issue Jan 5, 2024
* feat(button): adds pending button, fixes #3162

* chore(button): import sp-progress-circle whenever required

* chore: updated golden image hash

* chore(button): updated aria-label on changing pending state

* chore(button): removed redudant styles and functions

* chore(button): added tshirt sizing to the progress-circle

* chore(button): removed isPending state

* chore(button): update aria-label on pending toggle

* chore: updated golden image hash

* chore(button): removed ispending and timeout

* chore(button): updated css animation names

* chore(button): update aria-label on removing pending

* chore(button): added docs for pending state

* chore: updated golden image hash

* chore(button): update arialabel on disabled change in pending state

* chore: updated golden image hash

* fix(button): updated aria-label management in button

* fix(button): fixed aria-label management and added more tests

* fix(button): revert back the aria-label management logic in ButtonBase

* chore: updated golden image hash

* fix(button): shifted aria-label management to updated in button

* chore: udpated golden image hash

* chore: updated golden image hash

* chore(button): improved code readability

---------

Co-authored-by: TarunAdobe <ttomar@adobe.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Button enhancement New feature or request help wanted Extra attention is needed Spectrum Engineering
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants