Skip to content

Conversation

@JoaoFerreira-FrontEnd
Copy link
Contributor

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd commented Apr 2, 2024

  • here are the changes to have the checkbox small;

Issue number: resolves N/A


What is the current behavior?

What is the new behavior?

  • Adds the new small size to ionic theme checkbox
    -- The small size of the checkbox is 16px.

Does this introduce a breaking change?

  • Yes
  • No

- here are the changes to have the checkbox small;
@github-actions github-actions bot added package: core @ionic/core package package: angular @ionic/angular package package: vue @ionic/vue package labels Apr 2, 2024
Copy link
Contributor

@sean-perkins sean-perkins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small requested changes.

Do you need any assistance for generating screenshots locally for this PR?

@sean-perkins sean-perkins changed the title Small Checkbox for Ionic theme feat(checkbox): add small size Apr 2, 2024
Copy link
Contributor

@sean-perkins sean-perkins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good to me, great work. Just waiting on CI to pass. I restarted the job since the flaky alert dynamic scaling test is failing.

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd merged commit f79871a into sp/ROU-4822 Apr 2, 2024
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd deleted the fmj/ROU-4822-small branch April 2, 2024 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: angular @ionic/angular package package: core @ionic/core package package: vue @ionic/vue package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants