Skip to content
Snippets Groups Projects

Make Selection in Inkscape prettier

Merged Ishaan Arora requested to merge pulsar17/inkscape:prettier-selection-boxes into master

The scope of this MR has expanded to all the tools using selection. This should be possible with minor changes to this MR as most of the code is shared.

For a more complete picture of selection in various tools, one can refer to ux#239. The description below only covers two tools.

Addreses:

inbox#9940 (moved)

Inkscape currently has two selection boxes (the rectangle stroke that is drawn when the mouse is drag and dropped while using the Select tool):

  • Default
  • Touch

These look like the following (as of Inkscape 1.3.2 (091e20ef0f, 2023-11-25)):

mode screenshot
Default
Touch

The proposed designs after this change (with help from @adam.belis ):

mode screenshot
Default image
Touch image

The Node tool also has an additional "Lasso" mode which can be used select nodes by drawing a path around them while holding the Alt key. Current (as of Inkscape 1.3.2 (091e20ef0f, 2023-11-25))

node-tool-lasso

Proposed:

node-tool-lasso-proposed

(This "Lasso" mode is shared by Selector tool as well, although for Selector tool, it behaves like Touch selection box - any object the path touches is selected) For the Selector tool, the proposed design is just a dashed stroke without any fill or pattern. This is not yet implemented.

Edited by Adam Belis

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Mike Kowalski
  • Mike Kowalski
  • PBS
  • PBS
  • PBS
  • PBS
  • PBS
  • Ishaan Arora added 1 commit

    added 1 commit

    • f3718442 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Ishaan Arora added 3 commits

    added 3 commits

    • 70090569 - Use C++ interface of Cairo instead of C interface
    • 55aeb292 - Remove dashes from touch mode selection box
    • 312b015d - Add fill and fill pattern to touchpath selection

    Compare with previous version

  • Ishaan Arora changed the description

    changed the description

  • Adam Belis requested review from @adam.belis

    requested review from @adam.belis

  • Ishaan Arora mentioned in issue ux#239

    mentioned in issue ux#239

  • PBS
  • Ishaan Arora added 1 commit

    added 1 commit

    • 639bfcb4 - Fix data race by not setting the pattern matrix

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading