Skip to content

fix(agent): add scroll support to scheduled task dialog#366

Open
shadowfax92 wants to merge 1 commit intomainfrom
claude/slack-fix-dialog-scroll-issue-OBLCP
Open

fix(agent): add scroll support to scheduled task dialog#366
shadowfax92 wants to merge 1 commit intomainfrom
claude/slack-fix-dialog-scroll-issue-OBLCP

Conversation

@shadowfax92
Copy link
Contributor

Updates browseros-agent submodule with fix for dialog scroll issue. The scheduled task dialog now properly scrolls when content exceeds viewport.

Changes in submodule:

  • Added max-h-[90vh] and flex layout to DialogContent
  • Wrapped form fields in ScrollArea component
  • Fixed footer to bottom with flex-shrink-0

Slack: https://felafax.slack.com/archives/C0AF85H7PK8/p1771265964827929

https://claude.ai/code/session_01M2pUBV3Ld8oWx8CCEeTD7T

Updates browseros-agent submodule with fix for dialog scroll issue.
The scheduled task dialog now properly scrolls when content exceeds viewport.

Changes in submodule:
- Added max-h-[90vh] and flex layout to DialogContent
- Wrapped form fields in ScrollArea component
- Fixed footer to bottom with flex-shrink-0

Slack: https://felafax.slack.com/archives/C0AF85H7PK8/p1771265964827929

https://claude.ai/code/session_01M2pUBV3Ld8oWx8CCEeTD7T
@github-actions
Copy link
Contributor

CLA Assistant Lite bot Thank you for your submission! We require contributors to sign our Contributor License Agreement before we can accept your contribution.

By signing the CLA, you confirm that:

  • You have read and agree to the AGPL-3.0 license terms
  • Your contribution is your original work
  • You grant us the rights to use your contribution under the AGPL-3.0 license

To sign the CLA, please comment on this PR with:
I have read the CLA Document and I hereby sign the CLA


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@greptile-apps
Copy link

greptile-apps bot commented Feb 16, 2026

Greptile Summary

This PR updates the packages/browseros-agent submodule pointer (from 3b393f0 to 266402f) to pull in a UI fix for the scheduled task dialog. According to the PR description, the submodule changes add scroll support so the dialog properly scrolls when content exceeds the viewport.

  • Submodule pointer update: packages/browseros-agent advanced to include the dialog scroll fix
  • Underlying changes (per description): max-h-[90vh] and flex layout added to DialogContent, form fields wrapped in ScrollArea, footer fixed to bottom with flex-shrink-0
  • Note: The actual code changes live in the BrowserOS-agent submodule and cannot be directly reviewed from this parent repository. The submodule diff should be reviewed in the agent repo for completeness.

Confidence Score: 4/5

  • This PR is a low-risk submodule pointer update for a UI scroll fix with no logic or security implications in the parent repo.
  • The change is a single submodule pointer bump. The described changes are purely cosmetic/UI (adding scroll support to a dialog), which carries minimal risk. However, the actual code diff is not reviewable from this repo, so a perfect score isn't warranted — the submodule changes should be verified in the agent repository.
  • The actual code changes in packages/browseros-agent (the submodule) should be reviewed in the BrowserOS-agent repository directly, as they are not inspectable from the parent repo.

Important Files Changed

Filename Overview
packages/browseros-agent Submodule pointer update from 3b393f0 to 266402f. Per the PR description, the underlying changes add scroll support to the scheduled task dialog using max-h-[90vh], flex layout, ScrollArea, and flex-shrink-0 on the footer. The actual code diff is not reviewable from the parent repo.

Flowchart

flowchart TD
    A[Scheduled Task Dialog Opened] --> B{Content exceeds viewport?}
    B -->|No| C[Dialog renders normally]
    B -->|Yes| D[DialogContent applies max-h-90vh + flex layout]
    D --> E[ScrollArea wraps form fields]
    E --> F[Form fields become scrollable]
    D --> G[Footer fixed to bottom with flex-shrink-0]
    F --> H[User can scroll through fields and submit]
    G --> H
Loading

Last reviewed commit: 235b99b

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.

2 participants