Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(comp:alert): add alert component #520

Merged
merged 1 commit into from
Nov 16, 2021
Merged

Conversation

brenner8023
Copy link
Member

@brenner8023 brenner8023 commented Nov 14, 2021

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines
  • Tests for the changes have been added/updated or not needed
  • Docs and demo have been added/updated or not needed

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Component style update
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Application (the showcase website) / infrastructure changes
  • Other... Please describe:

What is the current behavior?

add Alert component

What is the new behavior?

IxAlert

Props

名称 说明 类型 默认值 全局配置 备注
closable 信息提示是否可关闭 boolean false -
closeIcon 自定义关闭按钮 string | #closeIcon close - -
description 辅助性文字介绍 string | #description - - -
icon 自定义图标 string | #icon - 若要隐藏图标则传空串
showPagination 是否开启分页切换效果 boolean false - -
type 设置提示类型 'success' | 'info' | 'warning' | 'error' info - -
title 信息提示内容 string | string[] | #default - - -
onBeforeClose 关闭提示前会触发的回调函数 () => boolean | Promise<boolean> - - -
onClose 关闭提示会触发的回调函数 () => void - - -

Other information

@idux-bot
Copy link

idux-bot bot commented Nov 14, 2021

This preview will be available after the AzureCI is passed.

@codecov
Copy link

codecov bot commented Nov 14, 2021

Codecov Report

Merging #520 (f05e60e) into main (6751b14) will increase coverage by 0.13%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #520      +/-   ##
==========================================
+ Coverage   84.12%   84.25%   +0.13%     
==========================================
  Files         292      294       +2     
  Lines        7370     7431      +61     
  Branches     1453     1473      +20     
==========================================
+ Hits         6200     6261      +61     
  Misses       1170     1170              
Impacted Files Coverage Δ
packages/components/alert/src/Alert.tsx 100.00% <100.00%> (ø)
packages/components/alert/src/types.ts 100.00% <100.00%> (ø)
packages/components/config/src/defaultConfig.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6751b14...f05e60e. Read the comment docs.

packages/components/alert/__tests__/alert.spec.ts Outdated Show resolved Hide resolved
packages/components/alert/__tests__/alert.spec.ts Outdated Show resolved Hide resolved
packages/components/alert/__tests__/alert.spec.ts Outdated Show resolved Hide resolved
packages/components/alert/__tests__/alert.spec.ts Outdated Show resolved Hide resolved
packages/components/alert/demo/Animation.vue Outdated Show resolved Hide resolved
packages/components/alert/src/Alert.tsx Outdated Show resolved Hide resolved
packages/components/alert/src/Alert.tsx Outdated Show resolved Hide resolved
packages/components/alert/src/types.ts Outdated Show resolved Hide resolved
packages/components/alert/style/mixin.less Outdated Show resolved Hide resolved
packages/components/alert/src/Alert.tsx Outdated Show resolved Hide resolved
@brenner8023 brenner8023 force-pushed the feat-alert branch 5 times, most recently from b0de663 to d9210da Compare November 16, 2021 00:28
@danranVm
Copy link
Member

需要解决一下 index.ts 的冲突,以及在 components.less 中,添加一下组件样式入口文件。

@danranVm danranVm mentioned this pull request Nov 16, 2021
95 tasks
@brenner8023 brenner8023 force-pushed the feat-alert branch 2 times, most recently from 279f3e4 to d64bd61 Compare November 16, 2021 04:40
@danranVm danranVm merged commit 38829ae into IDuxFE:main Nov 16, 2021
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