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.

Sign up for GitHub

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

Jump to bottom

include and exclude of keep alive component props are string should support spaces around commas #5873

Open
hchlq opened this issue May 6, 2022 · 1 comment · May be fixed by #5872
Open

include and exclude of keep alive component props are string should support spaces around commas #5873

hchlq opened this issue May 6, 2022 · 1 comment · May be fixed by #5872
Labels
🍰 p2-nice-to-have Priority 2: this is not breaking anything but nice to have it addressed. scope: keep-alive

Comments

@hchlq
Copy link
Contributor

hchlq commented May 6, 2022 •
edited
Loading

Version

3.2.33

Reproduction link

sfc.vuejs.org/

Steps to reproduce

  1. Switch to Bar component
  2. input something
  3. Switch to Foo component
  4. Switch to Bar component and the value of the input is missing

What is expected?

After input, switch to foo, and then switch to bar component. The value in the input should be retained

What is actually happening?

After input, switch to foo, and then switch to bar component. The value in the input is missing
@hchlq hchlq linked a pull request May 6, 2022 that will close this issue
Open
@hchlq hchlq changed the title the matcher whose include and exclude of keep alive component props are string should support spaces around commas include and exclude of keep alive component props are string should support spaces around commas May 7, 2022
@ByePastHub
Copy link

You need to bind v-model to the input, and the input value can be retained after switching

@LinusBorg LinusBorg added the 🍰 p2-nice-to-have Priority 2: this is not breaking anything but nice to have it addressed. label Jun 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🍰 p2-nice-to-have Priority 2: this is not breaking anything but nice to have it addressed. scope: keep-alive
Projects
None yet
Milestone
No milestone
4 participants
@LinusBorg @edison1105 @ByePastHub @hchlq