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

Making TagInput's Tag content customisable #2209

Merged
merged 4 commits into from
Feb 9, 2020

Conversation

jacobgoh101
Copy link
Contributor

Referring to #2208

Proposed Changes

  • Turning TagInput's Tag content into slot

@codecov
Copy link

codecov bot commented Feb 7, 2020

Codecov Report

Merging #2209 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev    #2209   +/-   ##
=======================================
  Coverage   65.14%   65.14%           
=======================================
  Files          70       70           
  Lines        2645     2645           
  Branches      868      868           
=======================================
  Hits         1723     1723           
  Misses        849      849           
  Partials       73       73
Impacted Files Coverage Δ
src/components/taginput/Taginput.vue 32% <ø> (ø) ⬆️

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 10ec092...8a8c51a. Read the comment docs.

@jacobgoh101 jacobgoh101 requested review from rafaberaldo and jtommy and removed request for rafaberaldo February 7, 2020 05:15
{{ getNormalizedTagText(tag) }}
<slot
name="tag"
:text="getNormalizedTagText(tag)"
Copy link
Member

@jtommy jtommy Feb 7, 2020

Choose a reason for hiding this comment

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

In this case I think we might remove :text because there is already tag parameter

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've removed it in the latest commit.

@jtommy jtommy merged commit 4fdbf49 into buefy:dev Feb 9, 2020
@jtommy
Copy link
Member

jtommy commented Feb 9, 2020

@jacobgoh101 thanks ;)

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