Skip to content

feat(components): [table-column] add filter-icon slot#17272

Merged
btea merged 11 commits intodevfrom
feat/table-add-filter-icon-prop
Jul 24, 2024
Merged

feat(components): [table-column] add filter-icon slot#17272
btea merged 11 commits intodevfrom
feat/table-add-filter-icon-prop

Conversation

@btea
Copy link
Copy Markdown
Member

@btea btea commented Jun 19, 2024

Please make sure these boxes are checked before submitting your PR, thank you!

  • Make sure you follow contributing guide English | (中文 | Español | Français).
  • Make sure you are merging your commits to dev branch.
  • Add some descriptions and refer to relative issues for your PR.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 19, 2024

@github-actions github-actions Bot added the CommitMessage::Qualified Qualified commit message label Jun 19, 2024
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 29, 2024

🧪 Playground Preview: https://element-plus.run/?pr=17272
Please comment the example via this playground if needed.

Comment thread docs/en-US/component/table.md Outdated
@btea btea requested review from kooriookami and ryuhangyeong July 1, 2024 13:39
@btea btea added the Project::Enhancement New feature or request label Jul 6, 2024
Comment thread packages/components/table/src/filter-panel.vue Outdated
@btea btea changed the title feat(components): [table] add filter-icon prop feat(components): [table-column] add filter-icon slot Jul 23, 2024
Copy link
Copy Markdown
Member

@tolking tolking left a comment

Choose a reason for hiding this comment

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

Looks great.

Perhaps we can add a test case for it.

@btea
Copy link
Copy Markdown
Member Author

btea commented Jul 24, 2024

OK, it's been added.

Comment thread packages/components/table/__tests__/table.test.ts
@btea btea merged commit c7dea71 into dev Jul 24, 2024
@btea btea deleted the feat/table-add-filter-icon-prop branch July 24, 2024 06:47
@element-bot element-bot mentioned this pull request Jul 26, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CommitMessage::Qualified Qualified commit message Project::Enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants