Skip to content

feat(pkg.icons): add ActivityLine icon#89

Open
Vampika wants to merge 3 commits into
mainfrom
163
Open

feat(pkg.icons): add ActivityLine icon#89
Vampika wants to merge 3 commits into
mainfrom
163

Conversation

@Vampika

@Vampika Vampika commented Jul 16, 2026

Copy link
Copy Markdown

Добавлена иконка Activity icon для использования её в задаче xi.tutor(modules.board) #163

@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
xi-storybook Ready Ready Preview, Comment Jul 22, 2026 4:13pm

@unknownproperty unknownproperty left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Нужно также поднять версию пакета иконок, убедиться, что она нормально отрисована в Storybook и сделать npm i в корне репозитория


export const ActivityLine = ({ ...props }: IconProps) => (
<Svg {...props}>
<path d="M22 12H18L15 21L9 3L6 12H2" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Мы переводим иконки из stroke в fill формат иначе не будут корректно работать стили для задания цветов через css

https://iconly.io/tools/svg-convert-stroke-to-fill

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Готово

@unknownproperty unknownproperty left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Не вижу обновлённого package-lock

@@ -1,5 +1,6 @@
import { Account } from './icons/Account';
import { Activity } from './icons/Activity';
import {ActivityLine} from './icons/ActivityLine'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Форматирование сломано

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants