Skip to content

Commit

Permalink
refactor: use shape=“round” instead
Browse files Browse the repository at this point in the history
  • Loading branch information
liamdebeasi committed Mar 21, 2024
1 parent e7248cf commit 6352186
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 10 deletions.
1 change: 0 additions & 1 deletion core/api.txt
Original file line number Diff line number Diff line change
Expand Up @@ -612,7 +612,6 @@ ion-input-password-toggle,prop,color,"danger" | "dark" | "light" | "medium" | "p
ion-input-password-toggle,prop,hideIcon,string | undefined,undefined,false,false
ion-input-password-toggle,prop,mode,"ios" | "md",undefined,false,false
ion-input-password-toggle,prop,showIcon,string | undefined,undefined,false,false
ion-input-password-toggle,part,button

ion-item,shadow
ion-item,prop,button,boolean,false,false,false
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -121,15 +121,11 @@ export class InputPasswordToggle implements ComponentInterface {
[mode]: true,
})}
>
{/*
This part is intentionally undocumented. It only exists so that Input
can style the button when InputPasswordToggle is slotted inside of the Input.
*/}
<ion-button
part="button"
mode={mode}
color={color}
fill="clear"
shape="round"
aria-checked={isPasswordVisible ? 'true' : 'false'}
aria-label="show password"
role="switch"
Expand Down
3 changes: 1 addition & 2 deletions core/src/components/input/input.ios.scss
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,7 @@
* don't look too crowded.
*/
::slotted(ion-button[slot="start"].button-has-icon-only),
::slotted(ion-button[slot="end"].button-has-icon-only),
::slotted(ion-input-password-toggle::part(button)) {
::slotted(ion-button[slot="end"].button-has-icon-only) {
--border-radius: 50%;
--padding-start: 0;
--padding-end: 0;
Expand Down
3 changes: 1 addition & 2 deletions core/src/components/input/input.md.scss
Original file line number Diff line number Diff line change
Expand Up @@ -115,8 +115,7 @@
* don't look too crowded.
*/
::slotted(ion-button[slot="start"].button-has-icon-only),
::slotted(ion-button[slot="end"].button-has-icon-only),
::slotted(ion-input-password-toggle::part(button)) {
::slotted(ion-button[slot="end"].button-has-icon-only) {
--border-radius: 50%;
--padding-start: 8px;
--padding-end: 8px;
Expand Down

0 comments on commit 6352186

Please sign in to comment.