-
Notifications
You must be signed in to change notification settings - Fork 789
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 scroll area component #945
Conversation
…src/' <!-- ELLIPSIS_HIDDEN --> > [!IMPORTANT] > Add `ScrollArea` component using Radix UI and update `PromptBox` to use it for displaying example prompts. > > - **Components**: > - Add `ScrollArea` and `ScrollBar` components in `scroll-area.tsx` using `@radix-ui/react-scroll-area`. > - **Dependencies**: > - Add `@radix-ui/react-scroll-area` to `package.json`. > - **Usage**: > - Update `PromptBox` in `PromptBox.tsx` to use `ScrollArea` for example prompts display. > > <sup>This description was created by </sup>[<img alt="Ellipsis" src="https://img.shields.io/badge/Ellipsis-blue?color=175173">](https://www.ellipsis.dev?ref=Skyvern-AI%2Fskyvern-cloud&utm_source=github&utm_medium=referral)<sup> for 11e01de2d4e911f160f735fad5584f75cabde640. It will automatically update as commits are pushed.</sup> <!-- ELLIPSIS_HIDDEN -->
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Incremental review on 3d6d9f3 in 8 seconds
More details
- Looked at
105
lines of code in2
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. skyvern-frontend/src/routes/tasks/create/PromptBox.tsx:13
- Draft comment:
ScrollBar
is already used withinScrollArea
, so importing it separately is unnecessary. Consider removing it from the import statement. - Reason this comment was not posted:
Confidence changes required:50%
The ScrollBar component is used within the ScrollArea component, so it doesn't need to be imported separately in PromptBox.tsx.
Workflow ID: wflow_ZTROVDEFUvy61L4v
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Reviewed everything up to 3d6d9f3 in 7 seconds
More details
- Looked at
105
lines of code in2
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. skyvern-frontend/src/routes/tasks/create/PromptBox.tsx:13
- Draft comment:
ScrollBar
is already included withinScrollArea
, so importingScrollBar
separately is unnecessary. Consider removing it from the import statement. - Reason this comment was not posted:
Confidence changes required:50%
The ScrollBar component is used within the ScrollArea component, so it doesn't need to be imported separately in PromptBox.tsx.
Workflow ID: wflow_IrQbdkROWNp5fTJK
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Incremental review on a5332b4 in 7 seconds
More details
- Looked at
12
lines of code in1
files - Skipped
1
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. skyvern-frontend/package.json:32
- Draft comment:
Ensure that@radix-ui/react-scroll-area
is actually used in the codebase to avoid unnecessary dependencies. - Reason this comment was not posted:
Confidence changes required:33%
The addition of the new dependency is correct and aligns with the PR description.
Workflow ID: wflow_V7B1qPj3Uov93rX6
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
Important
Add
ScrollArea
component using Radix UI and integrate it intoPromptBox
for displaying example prompts.ScrollArea
andScrollBar
components inscroll-area.tsx
using@radix-ui/react-scroll-area
.@radix-ui/react-scroll-area
topackage.json
.PromptBox
inPromptBox.tsx
to useScrollArea
for example prompts display.This description was created by for a5332b4. It will automatically update as commits are pushed.