Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

fix: added default slots documentation #561" #622

Merged
merged 4 commits into from
Dec 10, 2019

Conversation

kat10140
Copy link
Contributor

Related Issue: #561

Summary

@kat10140 kat10140 requested a review from a team as a code owner December 10, 2019 17:51
@kat10140 kat10140 self-assigned this Dec 10, 2019
@kat10140 kat10140 changed the title added default slots documentation #561" doc - added default slots documentation #561" Dec 10, 2019
@kat10140 kat10140 changed the title doc - added default slots documentation #561" fix: added default slots documentation #561" Dec 10, 2019
Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Nice

@kat10140 kat10140 merged commit 9ed8d74 into master Dec 10, 2019
@kat10140 kat10140 deleted the kat10140/doc-default-slots branch December 10, 2019 18:24
Copy link
Member

@jcfranco jcfranco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noice. 😎

@@ -10,6 +10,7 @@ import { CalciteTheme } from "../interfaces";
* @slot primary-panel - A slot for adding the leading `calcite-shell-panel`.
* @slot contextual-panel - A slot for adding the trailing `calcite-shell-panel`.
* @slot tip-manager - A slot for adding a `calcite-tip-manager`. This component will be absolutely positioned.
* @slot - A slot for adding content to the shell. This content will appear between any leading and trailing panels added to the shell. (eg. a map)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpick: ...shell. (eg. a map) -> shell (e.g. a map).

@@ -4,6 +4,9 @@ import { CSS } from "./resources";

import { CalciteTheme, FlowDirection } from "../interfaces";

/**
* @slot - A slot for adding `calcite-flow-items` to the flow.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpick: `calcite-flow-items` -> `calcite-flow-item`s

☝️ applies to other pluralized components.

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

Successfully merging this pull request may close these issues.

3 participants