Skip to content

Commit

Permalink
fix: update icon names to match names on figma
Browse files Browse the repository at this point in the history
  • Loading branch information
jongomez committed Oct 18, 2023
1 parent 356e8c5 commit 73ce937
Show file tree
Hide file tree
Showing 33 changed files with 71 additions and 97 deletions.
16 changes: 8 additions & 8 deletions packages/lsd-react/src/components/Calendar/Month.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ import { useRef, useState } from 'react'
import { useClickAway } from 'react-use'
import { IconButton } from '../IconButton'
import {
ArrowDownIcon,
ArrowUpIcon,
NavigateBeforeIcon,
NavigateNextIcon,
ChevronDownIcon,
ChevronUpIcon,
ChevronLeftIcon,
ChevronRightIcon,
} from '../Icons'
import { Typography } from '../Typography'
import { calendarClasses } from './Calendar.classes'
Expand Down Expand Up @@ -68,7 +68,7 @@ export const Month = ({
type="button"
onClick={goToPreviousMonths}
>
<NavigateBeforeIcon color="primary" />
<ChevronLeftIcon color="primary" />
</button>
<div className={calendarClasses.row}>
<Typography
Expand All @@ -92,13 +92,13 @@ export const Month = ({
onClick={() => setYear(year + 1)}
className={calendarClasses.changeYearButton}
>
<ArrowUpIcon color="primary" />
<ChevronUpIcon color="primary" />
</IconButton>
<IconButton
onClick={() => setYear(year - 1)}
className={calendarClasses.changeYearButton}
>
<ArrowDownIcon color="primary" />
<ChevronDownIcon color="primary" />
</IconButton>
</div>
</div>
Expand All @@ -117,7 +117,7 @@ export const Month = ({
type="button"
onClick={goToNextMonths}
>
<NavigateNextIcon color="primary" />
<ChevronRightIcon color="primary" />
</button>
</div>
<div className={clsx(calendarClasses.grid)}>
Expand Down
10 changes: 5 additions & 5 deletions packages/lsd-react/src/components/Checkbox/Checkbox.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,8 @@ import {
} from '../../utils/useCommonProps'
import { useInput } from '../../utils/useInput'
import { useCheckboxGroupContext } from '../CheckboxGroup/CheckboxGroup.context'
import { CheckboxFilledIcon, CheckboxIcon } from '../Icons'
import { CheckboxIndeterminateIcon } from '../Icons/CheckboxIndeterminate'
import { CheckboxIcon, CheckboxOutlineBlankIcon } from '../Icons'
import { IndeterminateCheckboxFilledIcon } from '../Icons/IndeterminateCheckboxFilledIcon'
import { Typography } from '../Typography'
import { checkboxClasses } from './Checkbox.classes'

Expand Down Expand Up @@ -98,11 +98,11 @@ export const Checkbox: React.FC<CheckboxProps> & {
{...inputProps}
/>
{indeterminate ? (
<CheckboxIndeterminateIcon color="primary" focusable={false} />
<IndeterminateCheckboxFilledIcon color="primary" focusable={false} />
) : input.value ? (
<CheckboxFilledIcon color="primary" focusable={false} />
) : (
<CheckboxIcon color="primary" focusable={false} />
) : (
<CheckboxOutlineBlankIcon color="primary" focusable={false} />
)}
<span className={checkboxClasses.label}>{children}</span>
</Typography>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import {
omitCommonProps,
useCommonProps,
} from '../../utils/useCommonProps'
import { ArrowDownIcon, ArrowUpIcon } from '../Icons'
import { ChevronDownIcon, ChevronUpIcon } from '../Icons'
import { Typography } from '../Typography'
import { collapseHeaderClasses } from './CollapseHeader.classes'

Expand Down Expand Up @@ -58,12 +58,12 @@ export const CollapseHeader: React.FC<CollapseHeaderProps> & {
</Typography>
<div className={collapseHeaderClasses.icons}>
{open ? (
<ArrowUpIcon
<ChevronUpIcon
color="primary"
className={collapseHeaderClasses.menuIcon}
/>
) : (
<ArrowDownIcon
<ChevronDownIcon
color="primary"
className={collapseHeaderClasses.menuIcon}
/>
Expand Down
6 changes: 3 additions & 3 deletions packages/lsd-react/src/components/Dropdown/Dropdown.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ import {
import { SelectOption, useSelect } from '../../utils/useSelect'
import { DropdownItem } from '../DropdownItem'
import { DropdownMenu } from '../DropdownMenu'
import { ArrowDownIcon, ArrowUpIcon, ErrorIcon } from '../Icons'
import { ChevronDownIcon, ChevronUpIcon, ErrorIcon } from '../Icons'
import { Portal } from '../PortalProvider/Portal'
import { Typography } from '../Typography'
import { dropdownClasses } from './Dropdown.classes'
Expand Down Expand Up @@ -138,12 +138,12 @@ export const Dropdown: React.FC<DropdownProps> & {
)}

{openState ? (
<ArrowUpIcon
<ChevronUpIcon
color="primary"
className={dropdownClasses.menuIcon}
/>
) : (
<ArrowDownIcon
<ChevronDownIcon
color="primary"
className={dropdownClasses.menuIcon}
/>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import {
omitCommonProps,
useCommonProps,
} from '../../utils/useCommonProps'
import { CheckboxFilledIcon, CheckboxIcon, LsdIconProps } from '../Icons'
import { CheckboxIcon, CheckboxOutlineBlankIcon, LsdIconProps } from '../Icons'
import { Typography } from '../Typography'
import { dropdownItemClasses } from './DropdownItem.classes'

Expand Down Expand Up @@ -52,9 +52,9 @@ export const DropdownItem: React.FC<DropdownItemProps> & {
>
{withIcon &&
(selected ? (
<CheckboxFilledIcon {...iconProps} />
) : (
<CheckboxIcon {...iconProps} />
) : (
<CheckboxOutlineBlankIcon {...iconProps} />
))}
<Typography
variant={size === 'large' ? 'label1' : 'label2'}
Expand Down

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,9 @@ export const CheckboxIcon = LsdIcon(
{...props}
>
<path
d="M11.0833 2.91667V11.0833H2.91667V2.91667H11.0833ZM11.0833 1.75H2.91667C2.275 1.75 1.75 2.275 1.75 2.91667V11.0833C1.75 11.725 2.275 12.25 2.91667 12.25H11.0833C11.725 12.25 12.25 11.725 12.25 11.0833V2.91667C12.25 2.275 11.725 1.75 11.0833 1.75Z"
fillRule="evenodd"
clipRule="evenodd"
d="M2.91667 1.75H11.0833C11.725 1.75 12.25 2.275 12.25 2.91667V11.0833C12.25 11.725 11.725 12.25 11.0833 12.25H2.91667C2.275 12.25 1.75 11.725 1.75 11.0833V2.91667C1.75 2.275 2.275 1.75 2.91667 1.75ZM9.68333 4.43333L10.5 5.25L5.83333 9.91667L3.5 7.58333L4.31667 6.76667L5.83333 8.28333L9.68333 4.43333Z"
fill="black"
/>
</svg>
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
import { LsdIcon } from '../LsdIcon'

export const CheckboxOutlineBlankIcon = LsdIcon(
(props) => (
<svg
width="14"
height="14"
viewBox="0 0 14 14"
fill="none"
xmlns="http://www.w3.org/2000/svg"
{...props}
>
<path
d="M11.0833 2.91667V11.0833H2.91667V2.91667H11.0833ZM11.0833 1.75H2.91667C2.275 1.75 1.75 2.275 1.75 2.91667V11.0833C1.75 11.725 2.275 12.25 2.91667 12.25H11.0833C11.725 12.25 12.25 11.725 12.25 11.0833V2.91667C12.25 2.275 11.725 1.75 11.0833 1.75Z"
fill="black"
/>
</svg>
),
{
filled: true,
},
)
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
export * from './CheckboxOutlineBlankIcon'
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { LsdIcon } from '../LsdIcon'

export const ArrowDownIcon = LsdIcon(
export const ChevronDownIcon = LsdIcon(
(props) => (
<svg
width="14"
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
export * from './ChevronDownIcon'
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { LsdIcon } from '../LsdIcon'

export const NavigateBeforeIcon = LsdIcon(
export const ChevronLeftIcon = LsdIcon(
(props) => (
<svg
width="14"
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
export * from './ChevronLeftIcon'
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { LsdIcon } from '../LsdIcon'

export const NavigateNextIcon = LsdIcon(
export const ChevronRightIcon = LsdIcon(
(props) => (
<svg
width="14"
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
export * from './ChevronRightIcon'
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { LsdIcon } from '../LsdIcon'

export const ArrowUpIcon = LsdIcon(
export const ChevronUpIcon = LsdIcon(
(props) => (
<svg
width="14"
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
export * from './ChevronUpIcon'
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { LsdIcon } from '../LsdIcon'

export const CheckboxIndeterminateIcon = LsdIcon(
export const IndeterminateCheckboxFilledIcon = LsdIcon(
(props) => (
<svg
width="14"
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
export * from './IndeterminateCheckboxFilledIcon'

This file was deleted.

This file was deleted.

This file was deleted.

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { LsdIcon } from '../LsdIcon'

export const RadioButtonFilledIcon = LsdIcon(
export const RadioButtonCheckedIcon = LsdIcon(
(props) => (
<svg
width="14"
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
export * from './RadioButtonCheckedIcon'

This file was deleted.

13 changes: 6 additions & 7 deletions packages/lsd-react/src/components/Icons/index.ts
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
export * from './AddIcon'
export * from './ArrowDownIcon'
export * from './ChevronDownIcon'
export * from './ArrowForwardIcon'
export * from './ArrowUpIcon'
export * from './CheckboxFilledIcon'
export * from './ChevronUpIcon'
export * from './CheckboxIcon'
export * from './CheckboxOutlineBlankIcon'
export * from './CheckIcon'
export * from './CloseIcon'
export * from './ErrorIcon'
Expand All @@ -12,12 +12,11 @@ export * from './KeyboardReturnIcon'
export * from './LsdIcon'
export * from './MenuIcon'
export * from './MoreIcon'
export * from './NavigateBeforeIcon'
export * from './NavigateNextIcon'
export * from './NewPageIcon'
export * from './ChevronLeftIcon'
export * from './ChevronRightIcon'
export * from './SearchIcon'
export * from './PickIcon'
export * from './RadioButtonIcon'
export * from './RadioButtonFilledIcon'
export * from './RadioButtonCheckedIcon'
export * from './CalendarIcon'
export * from './RemoveIcon'
4 changes: 2 additions & 2 deletions packages/lsd-react/src/components/RadioButton/RadioButton.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import {
useCommonProps,
} from '../../utils/useCommonProps'
import { useInput } from '../../utils/useInput'
import { RadioButtonFilledIcon, RadioButtonIcon } from '../Icons'
import { RadioButtonCheckedIcon, RadioButtonIcon } from '../Icons'
import { useRadioButtonGroupContext } from '../RadioButtonGroup/RadioButtonGroup.context'
import { Typography } from '../Typography'
import { radioButtonClasses } from './RadioButton.classes'
Expand Down Expand Up @@ -91,7 +91,7 @@ export const RadioButton: React.FC<RadioButtonProps> & {
{...inputProps}
/>
{input.value ? (
<RadioButtonFilledIcon color="primary" focusable={false} />
<RadioButtonCheckedIcon color="primary" focusable={false} />
) : (
<RadioButtonIcon color="primary" focusable={false} />
)}
Expand Down
6 changes: 3 additions & 3 deletions packages/lsd-react/src/components/Tabs/Tabs.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import {
useCommonProps,
} from '../../utils/useCommonProps'
import { useHorizontalScroll } from '../../utils/useHorizontalScroll'
import { NavigateBeforeIcon, NavigateNextIcon } from '../Icons'
import { ChevronLeftIcon, ChevronRightIcon } from '../Icons'
import { TabItem } from '../TabItem'
import { TabsContext } from './Tab.context'
import { tabsClasses } from './Tabs.classes'
Expand Down Expand Up @@ -69,7 +69,7 @@ export const Tabs: React.FC<TabsProps> & {
onClick={() => scroll.toLeft()}
className={tabsClasses.leftScrollControl}
>
<NavigateBeforeIcon color="primary" />
<ChevronLeftIcon color="primary" />
</TabItem>
)}
{children}
Expand All @@ -81,7 +81,7 @@ export const Tabs: React.FC<TabsProps> & {
onClick={() => scroll.toRight()}
className={tabsClasses.rightScrollControl}
>
<NavigateNextIcon color="primary" />
<ChevronRightIcon color="primary" />
</TabItem>
)}
</div>
Expand Down

0 comments on commit 73ce937

Please sign in to comment.