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

fix(button): change loading button contents from div to span (backport to next) #1190

Merged
merged 1 commit into from
Feb 2, 2024

Commits on Feb 2, 2024

  1. fix(button): change loading button conents from div to span (#1181)

    ## PR Checklist
    
    Please check if your PR fulfills the following requirements:
    
    - [ ] Tests for the changes have been added (for bug fixes / features)
    - [ ] Docs have been added / updated (for bug fixes / features)
    - [ ] If applicable, have a visual design approval
    
    ## PR Type
    
    What kind of change does this PR introduce?
    
    <!-- Please check the one that applies to this PR using "x". -->
    
    - [x] Bugfix
    - [ ] Feature
    - [ ] Code style update (formatting, local variables)
    - [ ] Refactoring (no functional changes, no api changes)
    - [ ] Build related changes
    - [ ] CI related changes
    - [ ] Documentation content changes
    - [ ] Other... Please describe:
    
    ## What is the current behavior?
    
    In #1141 the loading button contents were added into a div to prevent
    the button from animating on the initial render.
    
    Issue Number: CDE-1628
    
    ## What is the new behavior?
    
    The div is changed to a span because adding a div inside of a button is
    technically not semantically correct html.
    
    ## Does this PR introduce a breaking change?
    
    - [ ] Yes
    - [x] No
    
    <!-- If this PR contains a breaking change, please describe the impact
    and migration path for existing applications below. -->
    
    ## Other information
    
    (cherry picked from commit 24fe115)
    williamernest authored and github-actions[bot] committed Feb 2, 2024
    Configuration menu
    Copy the full SHA
    3bb14ce View commit details
    Browse the repository at this point in the history