Skip to content

Add View Guides checkbox to Rings control tab#1

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/add-view-guides-checkbox
Draft

Add View Guides checkbox to Rings control tab#1
Copilot wants to merge 3 commits intomainfrom
copilot/add-view-guides-checkbox

Conversation

Copy link
Copy Markdown

Copilot AI commented Nov 4, 2025

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>
Copy link
Copy Markdown

Copilot AI left a comment

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
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