Skip to content

Add View Guides checkbox to Rings control tab - #1

Draft
mxjxn with Copilot wants to merge 3 commits into
mainfrom
copilot/add-view-guides-checkbox
Draft

mxjxn with Copilot wants to merge 3 commits into
mainfrom
copilot/add-view-guides-checkbox

Conversation

Copilot AI commented Nov 4, 2025 •

Copy link
Copy Markdown

The View Guides checkbox was only accessible from other tabs, requiring unnecessary navigation to toggle guide visibility while adjusting ring parameters.

Changes

  • Added View Guides checkbox to GlobalControls section in RingsControls.tsx
  • Imported guidesVisible and setGuidesVisible from artwork store
  • Positioned after stroke width slider, consistent with other control tabs

Implementation

const handleGuidesToggle = (checked: boolean) => {
  setGuidesVisible(checked);
  props.requestRedraw();
};

// In GlobalControls render:
<div class="global-control-item">
  <label class="guides-checkbox">
    <input
      type="checkbox"
      checked={guidesVisible()}
      onChange={(e) => handleGuidesToggle(e.currentTarget.checked)}
    />
    View Guides
  </label>
</div>

Screenshot

View Guides checkbox in Rings tab

The checkbox appears in the global controls section, allowing users to toggle guides without switching tabs.

Original prompt

make sure that a view guides check box is visible in the rings control tab


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: mxjxn <943499+mxjxn@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


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

Co-authored-by: mxjxn <943499+mxjxn@users.noreply.github.com>
Copilot AI changed the title [WIP] Add visibility for view guides checkbox in rings control tab Add View Guides checkbox to Rings control tab Nov 4, 2025
Copilot AI requested a review from mxjxn November 4, 2025 02:10

This branch had an error being deployed

1 failed deployment
github-pages — e947ac15 Deployed Nov 4, 2025 by Copilot via deploy #29
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.

3 participants