Conversation
🦋 Changeset detectedLatest commit: dbb25ca The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
ArtieReus
left a comment
There was a problem hiding this comment.
Just putting it into that state so I can suggest you something before. I hope it is fine :)
packages/ui-components/src/components/AppShellProvider/AppShellProvider.component.tsx
Fixed
Show fixed
Hide fixed
packages/ui-components/src/components/PanelFooter/PanelFooter.component.tsx
Fixed
Show fixed
Hide fixed
ArtieReus
left a comment
There was a problem hiding this comment.
some foundings to review...
|
|
||
| /** A all-purpose date and time picker component. Highly configurable, based on Flatpickr. */ | ||
| /** | ||
| * A all-purpose date and time picker component. Highly configurable, based on Flatpickr. |
There was a problem hiding this comment.
“A all-purpose …” → should be “An all-purpose …”
| /** Whether the DateTimePicker input element allows direct user keyboard input. Default is `false`. | ||
| */ | ||
| allowInput?: boolean | ||
| /** Allows the preloading of an invalid date (e.g. a date that hass been disable by passing `disable`). When disabled, the field will be cleared if the provided date is invalid */ |
There was a problem hiding this comment.
“hass been disable” → should be “has been disabled”
| "@cloudoperators/juno-ui-components": patch | ||
| --- | ||
|
|
||
| Improve component descriptions and prop documentation in editor for better developer experience |
There was a problem hiding this comment.
This reads okay, but you might consider “in the editor” instead of “in editor”
| */ | ||
| className?: string | ||
| } | ||
| export interface DataGridCellProps extends HTMLAttributes<HTMLDivElement> { |
There was a problem hiding this comment.
DataGridCell.component.tsx: the diff shows export interface DataGridCellProps ... declared twice... is this correct?
| export const Textarea: React.FC<TextareaProps> = ({ | ||
| /** | ||
| * A controlled Text Input. | ||
| * Also covers email, telephone, password, url derivates. |
There was a problem hiding this comment.
“url derivates” → should be “URL derivatives”
| /** The title of the modal. This will be rendering as the heading of the modal, and the modal's `arial-labelledby` attribute will reference the title/heading element. If the modal does not have `title` or `heading`, use `ariaLabel` to provide an accessible name for the modal. */ | ||
| export interface ModalProps extends Omit<HTMLProps<HTMLDivElement>, "size" | "title"> { | ||
| /** | ||
| * The title of the modal. This will be rendering as the heading of the modal, and the modal's `arial-labelledby` attribute will reference the title/heading element. If the modal does not have `title` or `heading`, use `ariaLabel` to provide an accessible name for the modal. |
There was a problem hiding this comment.
arial-labelledby → should be aria-labelledby
| /** Whether the DateTimePicker input element allows direct user keyboard input. Default is `false`. | ||
| */ | ||
| allowInput?: boolean | ||
| /** Allows the preloading of an invalid date (e.g. a date that hass been disable by passing `disable`). When disabled, the field will be cleared if the provided date is invalid */ |
There was a problem hiding this comment.
/** Allows the preloading of an invalid date (e.g. a date that hass been disable by passing disable). ... /
could be:
/* Allows preloading an invalid date (e.g. a date that has been disabled by passing disable). ... */
| /** A handler to execute once the navigation item is clicked. Will render the item as a button element if passed */ | ||
| onClick?: React.MouseEventHandler<HTMLElement> | ||
| onClick?: MouseEventHandler<HTMLElement> | ||
| /** An optional technical identifier fort the tab. If not passed, the label will be used to identify the tab. NOTE: If value is passed, the value of the active tab MUST be used when setting the activeItem prop on the parent TabNavigation.*/ |
There was a problem hiding this comment.
** An optional technical identifier fort the tab. ... /
to
/* An optional technical identifier for the tab. ... */
Summary
Enhance documentation of all UI Components in the editor, providing descriptions for all components and their props
Changes Made
Before
After
For all components:
For all props:
Related Issues
Closes #1429
Checklist
PR Manifesto
Review the PR Manifesto for best practises.