Repository navigation
base red: the lane report batch toggle keeps the default cursor (Default cursor check) - #28887
Closed
teamleaderleo wants to merge 1 commit into
Closed
teamleaderleo wants to merge 1 commit into
teamleaderleo wants to merge 1 commit into
Conversation
#28749 added cursor: pointer to .cv-lane-batch__toggle, which fails the Default cursor check (scripts/cmux-next/check-default-cursor.py) in cmux-next checks on every PR. The toggle's all: unset already leaves the default cursor. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Agent-Session: https://cmux.com/deeplink/agent?id=aid-f71664ebfb6cf5206479
Bugbot needs on-demand usage enabledBugbot uses usage-based billing for this team and requires on-demand usage to be enabled. A team admin can enable on-demand usage in the Cursor dashboard. |
Collaborator
Author
This branch was previously deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Base red fix:
cmux-next checks(Default cursor check,scripts/cmux-next/check-default-cursor.py) fails on every PR since #28749 (517a276) addedcursor: pointerto.cv-lane-batch__toggleinLaneNoticeBatch.css:17.This drops that one line. The toggle's
all: unsetalready leaves the default cursor, which is the rule the check enforces ("change all pointers back to cursor-default"). Nothing else changes.Test:
python3 scripts/cmux-next/check-default-cursor.pyprintscheck-default-cursor: okat this head.No capture: the only change is the pointer shape while hovering one toggle. Browser and gallery screenshots never include the OS pointer, so a still before and after is pixel-identical and can't show it. The evidence is the check itself:
check-default-cursor.pyfails on base at 6831eb6 (LaneNoticeBatch.css:17cursor: pointer) and passes at this head. The default cursor is the settled rule the check enforces (Leo, 2026-10-30: "change all pointers back to cursor-default"), so this restores the expected cursor and doesn't regress one. The toggle's hover highlight, which is what a still could show, is unchanged.Agent session: agent session
🤖 Generated with Claude Code