Skip to content
This repository was archived by the owner on Nov 9, 2026. It is now read-only.

fix: emit hscroll for left and right scroll directions - #255

Open
FOWEPJF255 wants to merge 1 commit into
bytedance:mainfrom
FOWEPJF255:fix/scroll-horizontal-hscroll
Open

FOWEPJF255 wants to merge 1 commit into
bytedance:mainfrom
FOWEPJF255:fix/scroll-horizontal-hscroll

Conversation

@FOWEPJF255

Copy link
Copy Markdown

Summary

  • The computer-use prompt already accepts scroll directions up, down, left, and right, but parsing_response_to_pyautogui_code only emitted pyautogui.scroll for vertical directions.
  • left and right were dropped. They now emit pyautogui.hscroll, using the same sign convention as vertical scroll: positive right, negative left.
  • Scrolls that include a point still pass x and y. Vertical scroll is unchanged.

Test plan

  • python -m unittest tests.action_parser_test (4 passed, including test_scroll_left_and_right_emit_hscroll)
  • Confirm CI on this PR

@CLAassistant

CLAassistant commented Sep 21, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

The computer-use prompt already lists four scroll directions, but the parser only generated vertical scroll calls, so left and right actions were dropped.
@FOWEPJF255
FOWEPJF255 force-pushed the fix/scroll-horizontal-hscroll branch from 6438fb3 to dc3c16c Compare September 22, 2026 14:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants