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: add mention component #17586

Merged
merged 20 commits into from
Aug 2, 2024
Merged

feat: add mention component #17586

merged 20 commits into from
Aug 2, 2024

Conversation

Fuphoenixes
Copy link
Contributor

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.

Copy link

👋 @Fuphoenixes, thank you for contributing element-plus.

  • You can comment with /label Components:[component_name] to add a label for which component you are working on.
  • You may join our Discord for staying tuned.

Copy link

github-actions bot commented Jul 18, 2024

@github-actions github-actions bot added the CommitMessage::Qualified Qualified commit message label Jul 18, 2024
@Fuphoenixes
Copy link
Contributor Author

@element-plus/backers see see

Copy link

github-actions bot commented Jul 18, 2024

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

@kooriookami
Copy link
Member

It's a great component!

packages/components/mention/src/helper.ts Outdated Show resolved Hide resolved
packages/components/mention/src/helper.ts Outdated Show resolved Hide resolved
packages/components/mention/src/helper.ts Outdated Show resolved Hide resolved
Copy link
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.

The position of the dropdown is incorrect when using the prepare slot. Bug

packages/components/mention/src/mention-dropdown.vue Outdated Show resolved Hide resolved
packages/components/mention/src/helper.ts Outdated Show resolved Hide resolved
packages/components/mention/src/helper.ts Show resolved Hide resolved
packages/components/mention/index.ts Outdated Show resolved Hide resolved
packages/components/input/src/input.vue Show resolved Hide resolved
packages/components/mention/src/mention-dropdown.vue Outdated Show resolved Hide resolved
packages/components/mention/src/mention.vue Outdated Show resolved Hide resolved
packages/components/mention/src/mention.vue Outdated Show resolved Hide resolved
packages/components/mention/src/mention.vue Outdated Show resolved Hide resolved
packages/components/mention/src/mention.ts Outdated Show resolved Hide resolved
docs/en-US/component/mention.md Outdated Show resolved Hide resolved
packages/components/mention/src/mention.vue Outdated Show resolved Hide resolved
packages/components/mention/src/mention.vue Show resolved Hide resolved
docs/en-US/component/mention.md Outdated Show resolved Hide resolved
docs/en-US/component/mention.md Outdated Show resolved Hide resolved
docs/en-US/component/input.md Outdated Show resolved Hide resolved
docs/examples/mention/loading.vue Outdated Show resolved Hide resolved
packages/components/mention/src/mention.ts Show resolved Hide resolved
packages/components/mention/src/mention.ts Show resolved Hide resolved
@kooriookami kooriookami merged commit 0456c79 into dev Aug 2, 2024
9 checks passed
@kooriookami kooriookami deleted the feat/mention branch August 2, 2024 03:00
tolking pushed a commit that referenced this pull request Aug 2, 2024
* feat: add mention component

* fix: build error

* fix: build error

* feat: delete as a whole

* fix: update docs

* fix: update global.d.ts

* fix: update

* fix: update code

* fix: update code

* fix: update code

* fix: rename

* fix: update code

* fix: upload code

* fix: update code

* fix: fixed cursor position abnormality

* fix: update code

* fix: docs add avatar

* fix: tooltip position is wrong when placing on top

* feat: add overview icon

* fix: overview icon color
tolking pushed a commit that referenced this pull request Aug 2, 2024
* feat: add mention component

* fix: build error

* fix: build error

* feat: delete as a whole

* fix: update docs

* fix: update global.d.ts

* fix: update

* fix: update code

* fix: update code

* fix: update code

* fix: rename

* fix: update code

* fix: upload code

* fix: update code

* fix: fixed cursor position abnormality

* fix: update code

* fix: docs add avatar

* fix: tooltip position is wrong when placing on top

* feat: add overview icon

* fix: overview icon color
tolking pushed a commit that referenced this pull request Aug 2, 2024
* feat: add mention component

* fix: build error

* fix: build error

* feat: delete as a whole

* fix: update docs

* fix: update global.d.ts

* fix: update

* fix: update code

* fix: update code

* fix: update code

* fix: rename

* fix: update code

* fix: upload code

* fix: update code

* fix: fixed cursor position abnormality

* fix: update code

* fix: docs add avatar

* fix: tooltip position is wrong when placing on top

* feat: add overview icon

* fix: overview icon color
@element-bot element-bot mentioned this pull request Aug 9, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants