Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix(MdTable): fix double default slots #1731

Merged
merged 1 commit into from
May 14, 2018
Merged

fix(MdTable): fix double default slots #1731

merged 1 commit into from
May 14, 2018

Conversation

Samuell1
Copy link
Member

Simple table without v-model is rendering default slots 2 times.
Reproduction: https://codesandbox.io/s/1qy64o65j?module=App.vue

@marcosmoura marcosmoura merged commit 3c61bcf into dev May 14, 2018
@marcosmoura marcosmoura deleted the fix/MdTable/fix-slot branch May 14, 2018 17:36
marcosmoura added a commit that referenced this pull request May 15, 2018
@marcosmoura
Merge remote-tracking branch 'origin/dev'
914161a 
* origin/dev:
  test(MdSwitch): fix test script (#1739)
  test(MdCheckbox): fix test script (#1737)
  test(MdBadge): fix test script (#1736)
  fix(MdTable): fix table rendering default slot twice (#1731)
@fabioviana-hotmart
Copy link

This problem still occurs when it has the component md-table-pagination.

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@marcosmoura marcosmoura marcosmoura approved these changes

@VdustR VdustR VdustR approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Samuell1 @fabioviana-hotmart @marcosmoura @VdustR