-
Notifications
You must be signed in to change notification settings - Fork 415
feat(gantt): add theme adaptation capability #3931
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
base: develop
Are you sure you want to change the base?
feat(gantt): add theme adaptation capability #3931
Conversation
| dragReorder_size?: number; | ||
| }; | ||
|
|
||
| grid?: {}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
不应该侵入到vTable的代码
4174140 to
4716ddd
Compare
| } | ||
| //#region gantt | ||
| export interface GanttConstructorOptions { | ||
| theme?: any; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
类型定义不合适
| // vtable-gantt/src/ts-types/gantt-theme-define.ts | ||
|
|
||
| export interface IGanttTableThemeDefine { | ||
| grid?: Record<string, any>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
类型定义不合适
| const CONTAINER_ID = 'vTable'; | ||
|
|
||
| export function createTable() { | ||
| const records = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
demo加个切换主题的tab 各个主题都展示出来 把这个demo做好看一些 后面要放到官网上面
| // ...可继续扩展其他主题字段 | ||
| }; | ||
|
|
||
| export class TableTheme { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
命名不要那vtable的直接用
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
#3049
💡 Background and solution
📝 Changelog
☑️ Self-Check before Merge
🚀 Summary
copilot:summary
🔍 Walkthrough
copilot:walkthrough