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

keep-alive component (or other abstract component) ignores slot name #5888

Closed
wonderful-panda opened this issue Jun 15, 2017 · 1 comment
Closed
Labels

Comments

@wonderful-panda
Copy link

Version

2.3.4

Reproduction link

https://jsfiddle.net/f055yu68/

Steps to reproduce

Just open reproduction link

What is expected?

"foo-content" is shown at foo slot.

default-slot :  ???
foo-slot : foo content

What is actually happening?

"foo-content" is shown at default slot.
slot="foo" specification is just ignored

default-slot : foo content 
foo-slot : ???

slot name is stored in data, and lost at this line.

// abstract components do not keep anything

Is this an intentional behaviour ?

@yyx990803 yyx990803 added the bug label Jun 15, 2017
awamwang added a commit to awamwang/vue that referenced this issue Jun 28, 2017
* 'dev' of https://github.com/vuejs/vue:
  build: use cross-platform hook installation with shelljs
  build: move test config files into /test
  build: add script for generating release note
  build: add git commit message validation
  fix(v-model): use consistent behavior during IME composition for other text-like input types (fix vuejs#5902)
  simplify source with rest params
  fix slot resolved incorrect with abstract component (fix vuejs#5888) (vuejs#5895)
  test:improve reserved props test
  test:add bind object test
  fix:when using object syntax in v-bind, special attribute have no effect
  Be able to use string type index in array (vuejs#5889)
  Merge inject when extending a component (vuejs#5827)

# Conflicts:
#	.gitignore
#	src/core/util/options.js
@Oscar-ren
Copy link

hi, same the condition, what about scopedSlots?

here is the demo: https://jsfiddle.net/anyvh33j/

could append the scopedSlots in the abstract component?

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants