Skip to content

chore(ui): improve developer docs#1428

Open
guoda-puidokaite wants to merge 9 commits intomainfrom
guoda-ui-components-jsdocs
Open

chore(ui): improve developer docs#1428
guoda-puidokaite wants to merge 9 commits intomainfrom
guoda-ui-components-jsdocs

Conversation

@guoda-puidokaite
Copy link
Contributor

@guoda-puidokaite guoda-puidokaite commented Jan 27, 2026

Summary

Enhance documentation of all UI Components in the editor, providing descriptions for all components and their props

Changes Made

Before

Screenshot 2026-02-17 at 10 04 53 Screenshot 2026-02-17 at 10 05 00

After

For all components:

  • Display available props
  • Add description, explaining use case
  • Add component Storybook link
  • Add interface definition link

For all props:

  • Add description, including defaults
Screenshot 2026-02-17 at 10 06 43 Screenshot 2026-02-17 at 10 06 51

Related Issues

Closes #1429

Checklist

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • I have made corresponding changes to the documentation (if applicable).
  • My changes generate no new warnings or errors.
  • I have created a changeset for my changes.

PR Manifesto

Review the PR Manifesto for best practises.

@guoda-puidokaite guoda-puidokaite self-assigned this Jan 27, 2026
@guoda-puidokaite guoda-puidokaite requested review from a team and franzheidl as code owners January 27, 2026 10:27
@guoda-puidokaite guoda-puidokaite added documentation Improvements or additions to documentation package All tasks related to package under packages/ ui-components All tasks related to juno-ui-components library labels Jan 27, 2026
@changeset-bot
Copy link

changeset-bot bot commented Jan 27, 2026

🦋 Changeset detected

Latest commit: dbb25ca

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages
Name Type
@cloudoperators/juno-ui-components Patch
@cloudoperators/juno-app-carbon Patch
@cloudoperators/juno-app-doop Patch
@cloudoperators/juno-app-example Patch
@cloudoperators/juno-app-greenhouse Patch
@cloudoperators/juno-app-heureka Patch
@cloudoperators/juno-app-supernova Patch
@cloudoperators/juno-app-template Patch
@cloudoperators/juno-messages-provider Patch

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

@github-actions
Copy link
Contributor

github-actions bot commented Jan 27, 2026

PR Preview Action v1.6.3

🚀 View preview at
https://cloudoperators.github.io/juno/pr-preview/pr-1428/

Built to branch gh-pages at 2026-02-17 10:20 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

Copy link
Collaborator

@ArtieReus ArtieReus left a comment

Choose a reason for hiding this comment

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

Just putting it into that state so I can suggest you something before. I hope it is fine :)

@guoda-puidokaite guoda-puidokaite changed the title chore(ui): improve developer docs by adding component and prop descriptions chore(ui): improve developer docs Feb 17, 2026
Copy link
Collaborator

@ArtieReus ArtieReus left a comment

Choose a reason for hiding this comment

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

amazing!!!!

Copy link
Collaborator

@ArtieReus ArtieReus left a comment

Choose a reason for hiding this comment

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

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.
Copy link
Collaborator

Choose a reason for hiding this comment

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

“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 */
Copy link
Collaborator

Choose a reason for hiding this comment

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

“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
Copy link
Collaborator

Choose a reason for hiding this comment

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

This reads okay, but you might consider “in the editor” instead of “in editor”

*/
className?: string
}
export interface DataGridCellProps extends HTMLAttributes<HTMLDivElement> {
Copy link
Collaborator

Choose a reason for hiding this comment

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

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.
Copy link
Collaborator

Choose a reason for hiding this comment

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

“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.
Copy link
Collaborator

Choose a reason for hiding this comment

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

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 */
Copy link
Collaborator

Choose a reason for hiding this comment

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

/** 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.*/
Copy link
Collaborator

Choose a reason for hiding this comment

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

** An optional technical identifier fort the tab. ... /
to
/
* An optional technical identifier for the tab. ... */

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation package All tasks related to package under packages/ ui-components All tasks related to juno-ui-components library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Task](ui): improve developer docs by adding component and prop descriptions

2 participants