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

Feature/collapse transition #72

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

novlan1
Copy link
Contributor

@novlan1 novlan1 commented Jul 2, 2024

collapse 组件

  1. transition 动画
  2. type导出
  3. 左右图标 demo
  4. 手风琴长短变化

@duenyang
Copy link
Contributor

duenyang commented Jul 2, 2024

image npm run dev 时,有报错

@novlan1
Copy link
Contributor Author

novlan1 commented Jul 2, 2024

image npm run dev 时,有报错

npm install 额

Tencent/omi#891

@duenyang
Copy link
Contributor

duenyang commented Jul 2, 2024

image 1. 箭头指向不对。2. 箭头转换方向时没有过渡动画。

@novlan1
Copy link
Contributor Author

novlan1 commented Jul 4, 2024

image 1. 箭头指向不对。2. 箭头转换方向时没有过渡动画。

done

@novlan1
Copy link
Contributor Author

novlan1 commented Jul 4, 2024

image 嵌套demo

import { css, globalCSS } from 'omi';

import globalStye from '../_common/style/web/_global.less';
import themeStyle from '../_common/style/web/theme/_index.less';
Copy link
Contributor

Choose a reason for hiding this comment

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

不需要使用theme.less,已经在style中引入

@@ -1,4 +1,4 @@
import { bind, Component, signal, tag } from 'omi';
Copy link
Contributor

Choose a reason for hiding this comment

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

需要增加static propsType,在纯html环境下需要使用

@tag('t-collapse-panel')
export default class CollapsePanel extends Component<TdCollapsePanelProps> {
static isLightDom = true;
static css = [
`.t-collapse-panel__wrapper--border-less .t-collapse-panel__header {
Copy link
Contributor

Choose a reason for hiding this comment

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

class的t使用classPrefix替换

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.

None yet

2 participants