Skip to content

Comments

Add line drawing tool#52

Merged
Microck merged 1 commit intomainfrom
feat/line-tool
Feb 23, 2026
Merged

Add line drawing tool#52
Microck merged 1 commit intomainfrom
feat/line-tool

Conversation

@Microck
Copy link
Collaborator

@Microck Microck commented Feb 17, 2026

Description

Add a dedicated line drawing tool accessible via key 3.

Type of Change

  • New feature (non-breaking change which adds functionality)

Changes Made

  • Add line tool to tool selector with SVG icon
  • Implement line drawing logic in pointer-events.js
  • Add cursor preview support in brush-helper.js
  • Style tool icon in tools.css

Testing

  • Tested in Firefox

Test Configuration:

  • OS: Windows 10
  • Browser: Firefox

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have tested my changes thoroughly

Related Issues

N/A

Microck added a commit that referenced this pull request Feb 21, 2026
Includes keybindings for both line and rect tools to avoid merge conflicts with PR #52
@hingler
Copy link
Collaborator

hingler commented Feb 23, 2026

LGTM!

  • Tested line tool with various colors
  • Ensured that lines are incorporated into their respective swatches, and thus can be erased

Undo/redo actions are not working with lines atm, but history's frankly kinda busted right now and I can see that you called the right history-related functions, so I'll give it a pass. Your call though

let draggingGuide = null;
let guideSnap = false;

let textEntryOverlay = null;
Copy link
Collaborator

Choose a reason for hiding this comment

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

text code and guide code should not be in this PR

@Microck Microck merged commit bf8382a into main Feb 23, 2026
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