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

add all docs for release in November #528

Merged
merged 22 commits into from
Nov 14, 2024
Merged

Conversation

britt6612
Copy link
Collaborator

@britt6612 britt6612 commented Nov 8, 2024

src/screens/Calendar.js Outdated Show resolved Hide resolved

<Property name="calendar.small.range.round">
<Description>
The amount to round the corner days in range when small.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same question.


<Property name="calendar.small.range.round.end">
<Description>
The amount to round the corner days in range when small.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same


<Property name="calendar.small.range.round.start">
<Description>
The amount to round the corner days in range when small.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same


<Property name="calendar.medium.range.round">
<Description>
The amount to round the corner days in range when medium.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same


<Property name="calendar.large.range.round.start">
<Description>
The amount to round the corner days in range when large.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same


<Property name="calendar.large.range.round.end">
<Description>
The amount to round the corner days in range when large.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same

@@ -361,6 +361,22 @@ background: {
<Example defaultValue>"background-front"</Example>
</PropertyValue>
</Property>
<Property name="notification.critical.message.color">
<Description>The color of the message.</Description>
Copy link
Collaborator

Choose a reason for hiding this comment

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

These should be more specific (e.g. The color of the message text for critical status.)

src/screens/ToggleGroup.js Outdated Show resolved Hide resolved
src/screens/ToggleGroup.js Outdated Show resolved Hide resolved
src/screens/Button.js Outdated Show resolved Hide resolved
src/screens/Button.js Outdated Show resolved Hide resolved
src/screens/Calendar.js Outdated Show resolved Hide resolved
src/screens/Calendar.js Outdated Show resolved Hide resolved
src/screens/Calendar.js Outdated Show resolved Hide resolved
src/screens/Calendar.js Outdated Show resolved Hide resolved
src/screens/Calendar.js Outdated Show resolved Hide resolved
src/screens/Calendar.js Outdated Show resolved Hide resolved
src/screens/Calendar.js Outdated Show resolved Hide resolved
src/screens/Calendar.js Outdated Show resolved Hide resolved
src/screens/Calendar.js Outdated Show resolved Hide resolved
britt6612 and others added 14 commits November 13, 2024 09:59
Co-authored-by: Taylor Seamans <[email protected]>
Co-authored-by: Taylor Seamans <[email protected]>
Co-authored-by: Taylor Seamans <[email protected]>
Co-authored-by: Taylor Seamans <[email protected]>
Co-authored-by: Taylor Seamans <[email protected]>
Co-authored-by: Taylor Seamans <[email protected]>
Co-authored-by: Taylor Seamans <[email protected]>
Co-authored-by: Taylor Seamans <[email protected]>
Co-authored-by: Taylor Seamans <[email protected]>
Co-authored-by: Taylor Seamans <[email protected]>
Co-authored-by: Taylor Seamans <[email protected]>
@britt6612 britt6612 requested a review from taysea November 13, 2024 17:37
<Example>"any CSS size"</Example>
</PropertyValue>
<PropertyValue type="object">
<Example>{`{ size: "...", corner: "..." }`}</Example>
Copy link
Collaborator

Choose a reason for hiding this comment

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

We're having to copy and paste this a lot of times -- is there room to abstract it into reusable example?

@@ -361,6 +361,30 @@ background: {
<Example defaultValue>"background-front"</Example>
</PropertyValue>
</Property>
<Property name="notification.critical.message.color">
Copy link
Collaborator

Choose a reason for hiding this comment

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

It looks like we're missing the kind specific options for each status

notification.critical.global.message.color
notification.critical.toast.message.color

@britt6612 britt6612 requested a review from taysea November 13, 2024 19:19
@britt6612 britt6612 merged commit 045d51f into master Nov 14, 2024
4 checks passed
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