Skip to content

feat(toast): toast service implementation (#UIM-829) #731

Open
wants to merge 102 commits into
base: master
Choose a base branch
from

Conversation

vladimirverbitski
Copy link
Contributor

No description provided.

}

ngOnInit(): void {
this.intervalId = setTimeout(() => this.animationState = 'closing', this.toast.duration);
Copy link
Contributor

Choose a reason for hiding this comment

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

по возможности нужно избегать setTimout... в других компонентах есть примеры с запуском анимации, посмотри, возможно получиться улучшить.

packages/mosaic/toast/toast.ref.ts Outdated Show resolved Hide resolved
packages/mosaic/toast/toast.service.ts Outdated Show resolved Hide resolved
packages/mosaic/toast/toast.service.ts Outdated Show resolved Hide resolved
packages/mosaic/toast/toast.service.ts Outdated Show resolved Hide resolved
packages/mosaic/toast/toast.service.ts Outdated Show resolved Hide resolved
packages/mosaic/toast/toast.ref.ts Outdated Show resolved Hide resolved
packages/mosaic/toast/toast.service.ts Outdated Show resolved Hide resolved
packages/mosaic/toast/toast.service.ts Outdated Show resolved Hide resolved
packages/mosaic/toast/toast.type.ts Outdated Show resolved Hide resolved
@lskramarov lskramarov changed the title feat(toast): toast service implementation feat(toast): toast service implementation (#UIM-829) Feb 4, 2022
lskramarov and others added 6 commits February 7, 2022 16:39
…ID-1108

# Conflicts:
#	packages/docs/src/styles/default-theme/_variables.scss
#	packages/docs/src/styles/default-theme/css-tokens.css
#	packages/docs/src/styles/default-theme/tokens.ts
#	packages/docs/src/styles/green-theme/_variables.scss
#	packages/docs/src/styles/green-theme/css-tokens.css
#	packages/docs/src/styles/green-theme/tokens.ts
#	packages/docs/src/styles/red-theme/_variables.scss
#	packages/docs/src/styles/red-theme/css-tokens.css
#	packages/docs/src/styles/red-theme/tokens.ts
#	packages/docs/src/styles/yellow-theme/_variables.scss
#	packages/docs/src/styles/yellow-theme/css-tokens.css
#	packages/docs/src/styles/yellow-theme/tokens.ts
#	packages/mosaic-dev/toast/module.ts
#	packages/mosaic-dev/toast/template.html
#	packages/mosaic/design-tokens/legacy-2017/_variables.scss
#	packages/mosaic/design-tokens/legacy-2017/css-tokens.css
#	packages/mosaic/design-tokens/legacy-2017/tokens/components/checkbox.json5
#	packages/mosaic/design-tokens/legacy-2017/tokens/components/navbar.json5
#	packages/mosaic/design-tokens/legacy-2017/tokens/components/tags.json5
#	packages/mosaic/design-tokens/tokens.ts
#	packages/mosaic/toast/toast-container.component.ts
#	packages/mosaic/toast/toast.component.html
#	packages/mosaic/toast/toast.component.scss
#	packages/mosaic/toast/toast.component.ts
#	packages/mosaic/toast/toast.module.ts
#	packages/mosaic/toast/toast.service.ts
#	packages/mosaic/toast/toast.type.ts
@lskramarov lskramarov changed the title feat(toast): toast service implementation (#UIM-829) feat(toast): toast service implementation (#UIM-829) WIP Feb 9, 2022
@PositiveJS
Copy link
Contributor

Preview docs changes for b516a86 at https://positive-js.github.io/mosaic-previews/pr731-b516a86/

@lskramarov
Copy link
Contributor

@agalaktionov нужен список токенов или хотя бы примерное понимание какие параметры мы собираемся настраивать через токены. Так же обрати внимание на анимацию, возможно ее тоже нужно будет подправить.

@sonarcloud
Copy link

sonarcloud bot commented Feb 11, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 12 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@PositiveJS
Copy link
Contributor

Preview docs changes for 57a1695 at https://positive-js.github.io/mosaic-previews/pr731-57a1695/

@lskramarov lskramarov changed the title feat(toast): toast service implementation (#UIM-829) WIP feat(toast): toast service implementation (#UIM-829) Feb 11, 2022

export type McToastStyle = 'default' | 'confirm' | 'custom' | 'success' | 'error' | 'warning' | 'info';
export enum McToastPosition {
TOP_RIGHT = 'top-right',
Copy link
Member

Choose a reason for hiding this comment

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

@lskramarov возможно использовать позиции, которые использовались в popover?

@agalaktionov
Copy link
Contributor

@agalaktionov нужен список токенов или хотя бы примерное понимание какие параметры мы собираемся настраивать через токены. Так же обрати внимание на анимацию, возможно ее тоже нужно будет подправить.

@lskramarov гайд по тостам пока в работе, можно ли по временить пока с параметрами для токенов?

@pimenovoleg
Copy link
Member

@vladimirverbitski синхронизируй с master веткой

@vladimirverbitski
Copy link
Contributor Author

@vladimirverbitski синхронизируй с master веткой

сделано

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants