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

fix($theme-default): slots don't allow customization for Sidebar & Page (close: #1950) #1951

Merged
merged 1 commit into from
Oct 21, 2019

Conversation

softwarewright
Copy link
Contributor

@softwarewright softwarewright commented Oct 14, 2019

…slots

Summary

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Docs
  • Build-related changes
  • Other, please describe:

If changing the UI of default theme, please provide the before/after screenshot:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

  • Chrome
  • Firefox
  • Safari
  • Edge
  • IE

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature
  • Related documents have been updated
  • Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

@softwarewright softwarewright changed the title WIP fix(1950): use templates so that slots can be used for pages and … fix #1950: use templates so that slots can be used for pages and … Oct 16, 2019
@softwarewright softwarewright changed the title fix #1950: use templates so that slots can be used for pages and … fix($theme-default): use templates so that slots can be used fix(1950) Oct 16, 2019
@ulivz ulivz changed the title fix($theme-default): use templates so that slots can be used fix(1950) fix($theme-default): internal slots doesn't work (close: #1950) Oct 21, 2019
@ulivz ulivz changed the title fix($theme-default): internal slots doesn't work (close: #1950) fix($theme-default): slots do not allow customization for the sidebar or page (close: #1950) Oct 21, 2019
@ulivz ulivz changed the title fix($theme-default): slots do not allow customization for the sidebar or page (close: #1950) fix($theme-default): slots don't allow customization for sidebar or page (close: #1950) Oct 21, 2019
@ulivz ulivz changed the title fix($theme-default): slots don't allow customization for sidebar or page (close: #1950) fix($theme-default): slots don't allow customization for <Sidebar /> or <Page /> (close: #1950) Oct 21, 2019
@ulivz ulivz changed the title fix($theme-default): slots don't allow customization for <Sidebar /> or <Page /> (close: #1950) fix($theme-default): slots don't allow customization for Sidebar & Page (close: #1950) Oct 21, 2019
@ulivz ulivz merged commit 890e85d into vuejs:master Oct 21, 2019
@vue-bot
Copy link

vue-bot commented Oct 21, 2019

Hey @softwarewright, thank you for your time and effort spent on this PR, contributions like yours help make Vue better for everyone. Cheers! 💚

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.

3 participants