Fix overflow of default value in parameters

[wintonzheng]

Important

Add scrollable areas to WorkflowParameterAddPanel.tsx and WorkflowParameterEditPanel.tsx to handle content overflow.

• UI Enhancement:
  • Added ScrollArea and ScrollAreaViewport to WorkflowParameterAddPanel.tsx and WorkflowParameterEditPanel.tsx to handle content overflow.
  • Ensures UI remains usable with many parameters or long descriptions by making the panels scrollable.

^{This description was created by for d1f2f68. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Incremental review on d1f2f68 in 25 seconds

More details

• Looked at 1021 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. skyvern-frontend/src/routes/workflows/editor/panels/WorkflowParameterAddPanel.tsx:70

• Draft comment:
  Consider refactoring the repeated logic for handling different parameter types into separate functions or components to improve maintainability and reduce redundancy. This pattern is repeated in WorkflowParameterEditPanel as well.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The code in both WorkflowParameterAddPanel and WorkflowParameterEditPanel has a repeated pattern for handling different parameter types. This can be refactored to improve maintainability and reduce redundancy.

2. skyvern-frontend/src/routes/workflows/editor/panels/WorkflowParameterEditPanel.tsx:111

• Draft comment:
  Consider refactoring the repeated logic for handling different parameter types into separate functions or components to improve maintainability and reduce redundancy. This pattern is repeated in WorkflowParameterAddPanel as well.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The code in both WorkflowParameterAddPanel and WorkflowParameterEditPanel has a repeated pattern for handling different parameter types. This can be refactored to improve maintainability and reduce redundancy.

Workflow ID: wflow_Ja05yyhk08HzaOUc

---

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to d1f2f68 in 50 seconds

More details

• Looked at 1021 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. skyvern-frontend/src/routes/workflows/editor/panels/WorkflowParameterAddPanel.tsx:238

• Draft comment:
  Consider refactoring the JSON parsing logic into a utility function to avoid code repetition and improve readability. This pattern is repeated in WorkflowParameterEditPanel.tsx as well.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The code in both WorkflowParameterAddPanel.tsx and WorkflowParameterEditPanel.tsx has a repeated pattern for handling JSON default values. This can be refactored into a utility function to improve code readability and maintainability.

2. skyvern-frontend/src/routes/workflows/editor/panels/WorkflowParameterEditPanel.tsx:279

• Draft comment:
  Consider refactoring the JSON parsing logic into a utility function to avoid code repetition and improve readability. This pattern is repeated in WorkflowParameterAddPanel.tsx as well.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The code in both WorkflowParameterAddPanel.tsx and WorkflowParameterEditPanel.tsx has a repeated pattern for handling JSON default values. This can be refactored into a utility function to improve code readability and maintainability.

Workflow ID: wflow_ctYmSqU8y5YPVzDx

---

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