Skip to content

Add warning about scoped Slot implementation #1212

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

Closed
wants to merge 125 commits into from

Conversation

LinusBorg
Copy link
Member

No description provided.

@LinusBorg LinusBorg added the docs label Apr 19, 2019
correct typo

Co-Authored-By: LinusBorg <[email protected]>
@eddyerburgh
Copy link
Member

Thanks @LinusBorg . Can you run prettier to fix the formatting issue?

@LinusBorg
Copy link
Member Author

@eddyerburgh

  • when i run prettier on the whole repo (yarn format) it changes >200 files.
  • When I run it on just the docs file (yarn prettier --no-semi --single-quote --write "./docs/api/options") it also changes around 20 other things, mostly turning - into* ` for lists.

No idea why?

@winniehell
Copy link
Contributor

@LinusBorg Just a wild guess but maybe #1247 helps here. 🤔

JessicaSachs and others added 25 commits December 18, 2019 13:15
feat: Store selector in Wrapper.find() / .findAll()
Previously wrapper.trigger('keydown', { keyCode: 65 }) would trigger a keydown event with a keyCode
of 0. This commit fixes that bug.

PR closes #1285
PR closes #1295
Selecting already-selected radio buttons or checking already-checked
checkboxes should not trigger events.
Older versions of Vue use the `click` DOM event for updating
radio/checkbox `v-model`s.
…iling

fix: use the keyCodes provided in options from wrapper.trigger
fix: Ensure setChecked() and setSelected() only trigger DOM events when state is changed (fix #1339)
correct typo

Co-Authored-By: LinusBorg <[email protected]>
@lmiller1990
Copy link
Member

lmiller1990 commented Jan 3, 2020

I tried to rebase and encountered the same problem described above, but did not notice until pushing the commits. So I just made a fresh fork of dev and copied the content over. It's here: #1389.

Going to close this one and get the other one merged.

@lmiller1990 lmiller1990 closed this Jan 3, 2020
lmiller1990 added a commit that referenced this pull request Jan 3, 2020
 Add warning about scoped Slot implementation (#1212)
@lmiller1990 lmiller1990 deleted the LinusBorg-docs-scopedSlots-1 branch January 3, 2020 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.