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

Update the example prompts and add overflow #943

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

ykeremy
Copy link
Contributor

@ykeremy ykeremy commented Oct 9, 2024

Important

Update example prompts and add horizontal overflow handling in PromptBox.tsx.

  • Example Prompts:
    • Added new example prompts: "What is the top NYT bestseller?", "What is the top ranked football team?", "Find the top selling electrical connector on finditparts" in PromptBox.tsx.
  • UI Enhancements:
    • Added overflow-x-scroll to the example prompts container in PromptBox.tsx to handle horizontal overflow.
    • Added whitespace-nowrap to example prompt items to prevent text wrapping in PromptBox.tsx.

This description was created by Ellipsis for d97d64a. It will automatically update as commits are pushed.

…src/'

<!-- ELLIPSIS_HIDDEN -->

> [!IMPORTANT]
> Update example prompts and add horizontal overflow handling in `PromptBox.tsx`.
>
>   - **Example Prompts**:
>     - Added new example prompts: "What is the top NYT bestseller?", "What is the top ranked football team?", "Find the top selling electrical connector on finditparts" in `PromptBox.tsx`.
>   - **UI Enhancements**:
>     - Added `overflow-x-scroll` to the example prompts container in `PromptBox.tsx` to handle horizontal overflow.
>     - Added `whitespace-nowrap` to example prompt items to prevent text wrapping in `PromptBox.tsx`.
>
> <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 9cd38c31add4e6dd76d117589e4acce6f98658c2. It will automatically update as commits are pushed.</sup>

<!-- ELLIPSIS_HIDDEN -->
@ykeremy ykeremy added the sync label Oct 9, 2024
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a 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 d97d64a in 12 seconds

More details
  • Looked at 29 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. skyvern-frontend/src/routes/tasks/create/PromptBox.tsx:179
  • Draft comment:
    Consider adding a max-w class to limit the width of the container and prevent potential layout issues.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The use of overflow-x-scroll and whitespace-nowrap is appropriate for handling horizontal overflow and preventing text wrapping, respectively. However, the overflow-x-scroll class should be accompanied by a max-w class to ensure the container doesn't grow indefinitely, which could lead to layout issues.

Workflow ID: wflow_yngUmlX635ffDAfX


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a 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 d97d64a in 17 seconds

More details
  • Looked at 29 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. skyvern-frontend/src/routes/tasks/create/PromptBox.tsx:179
  • Draft comment:
    Consider adding overflow-y-hidden to prevent vertical scrollbars.
<div className="flex gap-4 overflow-x-scroll overflow-y-hidden rounded-sm bg-slate-elevation1 p-4">
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The use of overflow-x-scroll and whitespace-nowrap is appropriate for handling horizontal overflow and preventing text wrapping, respectively. However, the overflow-x-scroll class should be accompanied by overflow-y-hidden to prevent any potential vertical scrollbars from appearing, which could affect the UI aesthetics.

Workflow ID: wflow_LD8I0fQO27UMCJjM


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@msalihaltun msalihaltun merged commit a833017 into main Oct 9, 2024
2 checks passed
@msalihaltun msalihaltun deleted the salih/add-new-example-prompts branch October 9, 2024 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants