Skip to content

Remove cursor: pointer to be more native-like#140

Merged
cyanzhong merged 1 commit into
mainfrom
develop
Jun 23, 2026
Merged

Remove cursor: pointer to be more native-like#140
cyanzhong merged 1 commit into
mainfrom
develop

Conversation

@cyanzhong

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adjusts cursor styling in the preview UI to feel more native (less “web-like”) by removing explicit cursor: pointer usage on interactive elements.

Changes:

  • Removed explicit cursor: pointer styling from Quick Look segmented controls and the update pill.
  • Removed cursor: pointer styling from task list checkboxes and overrode the GitHub theme’s pointer cursor on enabled task list labels.
  • Regenerated the committed dist/ bundles to reflect the CSS updates.

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.

File Description
styles/quicklook.css Removes explicit pointer cursor from Quick Look segmented buttons.
styles/main.css Removes pointer cursor from task checkboxes/update pill; adds a label cursor override for enabled task items.
dist/markedit-preview.js Updates embedded CSS in the full bundle to match source CSS changes.
dist/lite/markedit-preview.js Updates embedded CSS in the lite bundle to match source CSS changes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread styles/main.css Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 4 changed files in this pull request and generated no new comments.

@cyanzhong cyanzhong merged commit d834676 into main Jun 23, 2026
2 checks passed
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