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

Updated Git documentation. #983

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

MBudreviciusBentley
Copy link
Contributor

Changes

Updated packages under ui directory README.md and package.jsonfiles.

Testing

N/A

@MBudreviciusBentley MBudreviciusBentley requested a review from a team as a code owner August 26, 2024 13:30
@CLAassistant
Copy link

CLAassistant commented Aug 26, 2024

CLA assistant check
All committers have signed the CLA.

@MBudreviciusBentley MBudreviciusBentley self-assigned this Aug 26, 2024
@mayank99 mayank99 linked an issue Aug 26, 2024 that may be closed by this pull request
Comment on lines +14 to +31
import {
ContextMenu,
ContextMenuItem,
ContextSubMenu,
} from "@itwin/core-react";

export const Basic: Story = {
render: (props) => {
return (
<ContextMenu {...props}>
<ContextSubMenu label="Label 1" id="1">
<ContextMenuItem>Label 1</ContextMenuItem>
</ContextSubMenu>
</ContextMenu>
);
},
};

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
import {
ContextMenu,
ContextMenuItem,
ContextSubMenu,
} from "@itwin/core-react";
export const Basic: Story = {
render: (props) => {
return (
<ContextMenu {...props}>
<ContextSubMenu label="Label 1" id="1">
<ContextMenuItem>Label 1</ContextMenuItem>
</ContextSubMenu>
</ContextMenu>
);
},
};
import {
ContextMenu,
ContextMenuItem,
} from "@itwin/core-react";
export function App() {
return (
<ContextMenu>
<ContextMenuItem>Item 1</ContextMenuItem>
</ContextMenu>
);
};

Let's avoid storybook-specific types.

Copy link
Collaborator

@GerardasB GerardasB Sep 3, 2024

Choose a reason for hiding this comment

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

I think we might need separate setup and usage sections for each package...


See the [iTwin.js](https://www.itwinjs.org/learning/ui/core/) documentation for more information.
To get started with React components from the `@itwin/core-react` package, simply import the necessary components and utilities:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
To get started with React components from the `@itwin/core-react` package, simply import the necessary components and utilities:
To get started with the `@itwin/components-react` package, simply import the necessary components and utilities:

## Documentation
## Usage

To get started with React components from the `@itwin/imodel-components-react` package, simply import the necessary components and utilities:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
To get started with React components from the `@itwin/imodel-components-react` package, simply import the necessary components and utilities:
To get started with the `@itwin/imodel-components-react` package, simply import the necessary components and utilities:

};
```

For more details, check out the [@itwin/imodel-components-react](https://www.itwinjs.org/reference/imodel-components-react/).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
For more details, check out the [@itwin/imodel-components-react](https://www.itwinjs.org/reference/imodel-components-react/).
For more details, check out the [@itwin/imodel-components-react documentation](https://www.itwinjs.org/reference/imodel-components-react/).

@@ -1,7 +1,7 @@
{
"name": "@itwin/imodel-components-react",
"version": "4.17.0-dev.0",
"description": "iTwin.js UI IModel Components",
"description": "A react component library for iTwin.js UI IModel Components",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"description": "A react component library for iTwin.js UI IModel Components",
"description": "A react component library of iTwin.js UI iModel components",

Comment on lines +14 to +42
import {
PropertyDescription,
StandardTypeNames,
} from "@itwin/appui-abstract";
import {
PropertyFilterRuleGroupOperator,
PropertyFilter,
} from "@itwin/components-react";

function createProperties(): PropertyDescription {
return new PropertyDescription({
name: "string-prop",
displayLabel: "String Property",
typename: StandardTypeNames.String,
});
}

function createInitialFilter(): PropertyFilter {
const property = createProperty();
return {
operator: PropertyFilterRuleGroupOperator.Or,
rules: [
{
property: property,
operator: "like",
},
],
};
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Update this snippet to pass in some minimal data/props to PropertyFilterBuilderRenderer.

}
```

For more details, check out the [@itwin/appui-react documentation](https://www.itwinjs.org/reference/appui-react/).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
For more details, check out the [@itwin/appui-react documentation](https://www.itwinjs.org/reference/appui-react/).
For more details, check out [Get Started](https://www.itwinjs.org/ui/appui/get-started/) guide or [@itwin/appui-react documentation](https://www.itwinjs.org/reference/appui-react/).

@@ -1,7 +1,7 @@
{
"name": "@itwin/core-react",
"version": "4.17.0-dev.0",
"description": "iTwin.js UI core components",
"description": "A react component library for iTwin.js UI general purpose components",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"description": "A react component library for iTwin.js UI general purpose components",
"description": "A react component library of iTwin.js UI general purpose components",

```tsx
import { BaseSolarDataProvider } from "@itwin/imodel-components-react";

export const Basic: Story = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's avoid storybook specific snippets.

@@ -1,7 +1,7 @@
{
"name": "@itwin/components-react",
"version": "4.17.0-dev.0",
"description": "iTwin.js UI complex components",
"description": "A react component library for iTwin.js UI data-oriented components",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"description": "A react component library for iTwin.js UI data-oriented components",
"description": "A react component library of iTwin.js UI data-oriented components",

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.

Update outdated package.json fields and README.md
3 participants