Skip to content

feat(utils): add typical animations - #148

Open
oostap1985 wants to merge 10 commits into
masterfrom
feat/68555909_utilities-typical-animations
Open

oostap1985 wants to merge 10 commits into
masterfrom
feat/68555909_utilities-typical-animations

Conversation

@oostap1985

Copy link
Copy Markdown
Collaborator

Привет.
Это пробный коммит, чтобы понять, правильно ли я понял задачу.

Единственное, после сборки npm run build:core , не получилось протестировать. В файле test/web.html добавлял div c утилитами, но они не отображаются в стилях в браузере. Скорее всего я туплю и нужно как-то скомпилировать их было.

@oostap1985
oostap1985 requested a review from mr150 August 18, 2026 11:42
@mr150

mr150 commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

они не отображаются в стилях в браузере

И не должны. Там на страницу в <output> выводится сгенерированный CSS. Можно в отдельном коммите добавить, чтобы в <style> они выводились

@oostap1985 oostap1985 changed the title add animation util mSp add animation utils Aug 31, 2026
@oostap1985 oostap1985 changed the title add animation utils feat(utils): add typical animations Aug 31, 2026
@oostap1985
oostap1985 force-pushed the feat/68555909_utilities-typical-animations branch from ee228d3 to 643414d Compare September 3, 2026 15:17
'': content,
'c': content,
'f': fixed,
'ih': inherit,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Дефолтные ключевые слова CSS и так работают для всех утилит (во всех стандартных типах конвертации)

'': none,
),
),
'Fds': (

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Правильное сокращение тут будет Fils. По алгоритму нельзя перескакивать через буквы

@oostap1985
oostap1985 requested a review from mr150 October 3, 2026 15:21
@oostap1985
oostap1985 force-pushed the feat/68555909_utilities-typical-animations branch from ddc75e9 to c97160b Compare October 7, 2026 16:22

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Для этой утилиты пример не нужен

Comment thread test/web.html

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Это больше не нужно, учитывая, что в доке будут примеры

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Некорректный пример - тут должен быть именно шаблон, в который подставляются данные из доки. Изучи вот этот пример и посмотри, как он в доке отображается

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Зачем отдельный файл, если у нас есть packages/core/src/sass/css/styles/animation.scss?


Helper tools for creating basic animations — such as rotation, notification, "heartbeat," "breathing," and bounce—implemented using the `@keyframes` directive and the corresponding values ​​for the `animation` property.

.An-mSp — spin

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Никаких длинных - не надо они все ломают. И сейчас пример в доке сломан

}
}

.animation-place {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Здесь не должно быть стилей для доки. Они обычно тут пишутся. И большая часть утилит, типа D-f есть в css бандле, нет только каких-то произвольных, типа P11. В справочнике утилит и групп, есть пометка "CSS library". Под ней как раз стоят значения, которые есть в css бандле

This branch has not been deployed

No deployments
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