The Inbox: Morphic-kfr.1620.mcz

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

The Inbox: Morphic-kfr.1620.mcz

commits-2
A new version of Morphic was added to project The Inbox:
http://source.squeak.org/inbox/Morphic-kfr.1620.mcz

==================== Summary ====================

Name: Morphic-kfr.1620
Author: kfr
Time: 4 February 2020, 5:55:31.268512 pm
UUID: 741c35fd-d7ac-e54b-9199-ed1dda9e4df9
Ancestors: Morphic-kfr.1619

Enhancement for PolygonMorph. When a vertex is dropped it will snap if it is close to another PolygonMorphs vertex. It is possible to toggle functionality on/ off in menu

=============== Diff against Morphic-kfr.1619 ===============

Item was changed:
  BorderedMorph subclass: #PolygonMorph
+ instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm snapToOtherPolygons'
- instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm'
  classVariableNames: ''
  poolDictionaries: ''
  category: 'Morphic-Basic'!
 
  !PolygonMorph commentStamp: 'md 2/24/2006 20:34' prior: 0!
  This class implements a morph which can behave as four different objects depending on the the following two facts:
  - is it OPEN or CLOSED?
  - is it SEGMENTED or SMOOTHED.
 
  1. The OPEN and SEGMENTED variant looks like polyline.
 
  2. The OPEN and SMOOTHED variant looks like spline (kind of curve)
 
  3. The CLOSED and SEGMENTED variant looks like polygon. This is actually what you get when you do
  PolygonMorph new openInWorld
  You get a triangle. See below how to manipulate these objects...
 
  4. The CLOSED and SMOOTHED variant looks like blob (???)
 
  Prototypes of this morph can also be found in "Object Catalog". Several (different variants) of this object are among "Basic" morphs.
 
  Explore the assiciated morph-menu. It enables you
  - to toggle showing of "handles". They make it possible to
  - reposition already existing vertices (by moving yellow handles)
  - create new vertices (by moving green handles)
  - delete already existing vertices (by dragging and dropping one yellow handle closely
   nearby the adjacent yellow handle
    Handles can be made visible/hidden by shift+leftclicking the morph. This way it is possible
    to quickly show handles, adjust vertices and then again hide handles.
  - making closed polygon open, i.e. converting it to a curve (and vice versa)
  - toggle smoothed/segmented line/outline
  - set up custom dashing (for line, curves or borders of closed polygons
  - set up custom arrow-heads (for lines resp. curves)
 
  ------------------------------------------------------------------------------------------
  Implementation notes:
 
  This class combines the old Polygon and Curve classes.
 
  The 1-bit fillForm to make display and containment tests reasonably fast.  However, this functionality is in the process of being supplanted by balloon capabilities, which should eventually provide anti-aliasing as well.
 
  wiz 7/18/2004 21:26
  s have made some changes to this class to
 
  1) correct some bugs associated with one vertex polygons.
 
  2) prepare for some enhancements with new curves.
 
  3) add shaping items to menu.!

Item was changed:
  ----- Method: PolygonMorph>>addCustomMenuItems:hand: (in category 'menu') -----
  addCustomMenuItems: aMenu hand: aHandMorph
  "Add morph-specific items to the given menu which was invoked by the given hand.  This method provides is invoked both from the halo-menu and from the control-menu regimes."
 
  super addCustomMenuItems: aMenu hand: aHandMorph.
  aMenu addUpdating: #handlesShowingPhrase target: self action: #showOrHideHandles.
  vertices size > 2 ifTrue:
  [aMenu addUpdating: #openOrClosePhrase target: self action: #makeOpenOrClosed].
 
  aMenu addUpdating: #smoothPhrase target: self action: #toggleSmoothing.
+ aMenu addUpdating: #snapToOtherPolygonPhrase target: self action: #toggleSnapToOtherPolygons.
  aMenu addLine.
  aMenu add: 'specify dashed line' translated action:  #specifyDashedLine.
 
  self isOpen ifTrue:
  [aMenu addLine.
  aMenu addWithLabel: '---' enablement: [self isOpen and: [arrows ~~ #none]] action:  #makeNoArrows.
  aMenu addWithLabel: '-->' enablement: [self isOpen and: [arrows ~~ #forward]] action:  #makeForwardArrow.
  aMenu addWithLabel: '<--' enablement: [self isOpen and: [arrows ~~ #back]] action:  #makeBackArrow.
  aMenu addWithLabel: '<->' enablement: [self isOpen and: [arrows ~~ #both]] action:  #makeBothArrows.
  aMenu add: 'customize arrows' translated action: #customizeArrows:.
  (self hasProperty: #arrowSpec)
  ifTrue: [aMenu add: 'standard arrows' translated action: #standardArrows]].!

Item was changed:
  ----- Method: PolygonMorph>>dropVertex:event:fromHandle: (in category 'editing') -----
+ dropVertex: ix event: evt fromHandle: handle
+ "Leave vertex in new position. If dropped ontop another vertex delete
+ this one.
- dropVertex: ix event: evt fromHandle: handle
- "Leave vertex in new position. If dropped ontop another vertex delete this one.
  Check for too few vertices before deleting. The alternative
+ is not pretty -wiz"
+ | p world |
- is not pretty -wiz"
- | p |
  p := vertices at: ix.
+
+ "snap the dropped vertex to a vertex of another PolygonMorph if it is in near proximity"
+ self isSnappingToOtherPolygons
+ ifTrue: [world := Project current world.
+ world submorphs
+ do: [:each | ((each respondsTo: #vertices)
+ and: [each ~= self])
+ ifTrue: [each vertices
+ do: [:otherMorphsVertex | (otherMorphsVertex dist: p)
+ < 3
+ ifTrue: [vertices at: ix put: otherMorphsVertex]]]]].
  (vertices size >= 2
  and: ["check for too few vertices before deleting. The alternative
  is not pretty -wiz"
  ((vertices atWrap: ix - 1)
  dist: p)
  < 3
  or: [((vertices atWrap: ix + 1)
  dist: p)
  < 3]])
  ifTrue: ["Drag a vertex onto its neighbor means delete"
+ self deleteVertexAt: ix].
- self deleteVertexAt: ix .].
  evt shiftPressed
  ifTrue: [self removeHandles]
  ifFalse: [self addHandles
  "remove then add to recreate"]!

Item was changed:
  ----- Method: PolygonMorph>>initialize (in category 'initialization') -----
  initialize
  "initialize the state of the receiver"
  super initialize.
  ""
  vertices := Array
  with: 5 @ 0
  with: 20 @ 10
  with: 0 @ 20.
  closed := true.
  smoothCurve := false.
  arrows := #none.
+ snapToOtherPolygons := false.
  self computeBounds!

Item was added:
+ ----- Method: PolygonMorph>>isSnappingToOtherPolygons (in category 'access') -----
+ isSnappingToOtherPolygons
+   ^snapToOtherPolygons ifNil:[ snapToOtherPolygons := false].
+  !

Item was added:
+ ----- Method: PolygonMorph>>toggleSnapToOtherPolygons (in category 'menu') -----
+ toggleSnapToOtherPolygons
+      ^snapToOtherPolygons := snapToOtherPolygons not!


Reply | Threaded
Open this post in threaded view
|

Re: The Inbox: Morphic-kfr.1620.mcz

Christoph Thiede

Nice idea :-)


But when I merge this commit, it misses #snapToOtherPolygonPhrase:




Btw, #snapToOtherPolyongs itself is also absent in my image. Are you depending on another inbox commit?


Best,

Christoph



Von: Squeak-dev <[hidden email]> im Auftrag von [hidden email] <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 17:56:00
An: [hidden email]
Betreff: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
A new version of Morphic was added to project The Inbox:
http://source.squeak.org/inbox/Morphic-kfr.1620.mcz

==================== Summary ====================

Name: Morphic-kfr.1620
Author: kfr
Time: 4 February 2020, 5:55:31.268512 pm
UUID: 741c35fd-d7ac-e54b-9199-ed1dda9e4df9
Ancestors: Morphic-kfr.1619

Enhancement for PolygonMorph. When a vertex is dropped it will snap if it is close to another PolygonMorphs vertex. It is possible to toggle functionality on/ off in menu

=============== Diff against Morphic-kfr.1619 ===============

Item was changed:
  BorderedMorph subclass: #PolygonMorph
+        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm snapToOtherPolygons'
-        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm'
         classVariableNames: ''
         poolDictionaries: ''
         category: 'Morphic-Basic'!
 
  !PolygonMorph commentStamp: 'md 2/24/2006 20:34' prior: 0!
  This class implements a morph which can behave as four different objects depending on the the following two facts:
  - is it OPEN or CLOSED?
  - is it SEGMENTED or SMOOTHED.
 
  1. The OPEN and SEGMENTED variant looks like polyline.
 
  2. The OPEN and SMOOTHED variant looks like spline (kind of curve)
 
  3. The CLOSED and SEGMENTED variant looks like polygon. This is actually what you get when you do
         PolygonMorph new openInWorld
  You get a triangle. See below how to manipulate these objects...
 
  4. The CLOSED and SMOOTHED variant looks like blob (???)
 
  Prototypes of this morph can also be found in "Object Catalog". Several (different variants) of this object are among "Basic" morphs.
 
  Explore the assiciated morph-menu. It enables you
  - to toggle showing of "handles". They make it possible to
         - reposition already existing vertices (by moving yellow handles)
         - create new vertices (by moving green handles)
         - delete already existing vertices (by dragging and dropping one yellow handle closely
           nearby the adjacent yellow handle
    Handles can be made visible/hidden by shift+leftclicking the morph. This way it is possible
    to quickly show handles, adjust vertices and then again hide handles.
  - making closed polygon open, i.e. converting it to a curve (and vice versa)
  - toggle smoothed/segmented line/outline
  - set up custom dashing (for line, curves or borders of closed polygons
  - set up custom arrow-heads (for lines resp. curves)
 
  ------------------------------------------------------------------------------------------
  Implementation notes:
 
  This class combines the old Polygon and Curve classes.
 
  The 1-bit fillForm to make display and containment tests reasonably fast.  However, this functionality is in the process of being supplanted by balloon capabilities, which should eventually provide anti-aliasing as well.
 
  wiz 7/18/2004 21:26
  s have made some changes to this class to
 
  1) correct some bugs associated with one vertex polygons.
 
  2) prepare for some enhancements with new curves.
 
  3) add shaping items to menu.!

Item was changed:
  ----- Method: PolygonMorph>>addCustomMenuItems:hand: (in category 'menu') -----
  addCustomMenuItems: aMenu hand: aHandMorph
         "Add morph-specific items to the given menu which was invoked by the given hand.  This method provides is invoked both from the halo-menu and from the control-menu regimes."
 
         super addCustomMenuItems: aMenu hand: aHandMorph.
         aMenu addUpdating: #handlesShowingPhrase target: self action: #showOrHideHandles.
         vertices size > 2 ifTrue:
                 [aMenu addUpdating: #openOrClosePhrase target: self action: #makeOpenOrClosed].
 
         aMenu addUpdating: #smoothPhrase target: self action: #toggleSmoothing.
+        aMenu addUpdating: #snapToOtherPolygonPhrase target: self action: #toggleSnapToOtherPolygons.
         aMenu addLine.
         aMenu add: 'specify dashed line' translated action:  #specifyDashedLine.
 
         self isOpen ifTrue:
                 [aMenu addLine.
                 aMenu addWithLabel: '---' enablement: [self isOpen and: [arrows ~~ #none]] action:  #makeNoArrows.
                 aMenu addWithLabel: '-->' enablement: [self isOpen and: [arrows ~~ #forward]] action:  #makeForwardArrow.
                 aMenu addWithLabel: '<--' enablement: [self isOpen and: [arrows ~~ #back]] action:  #makeBackArrow.
                 aMenu addWithLabel: '<->' enablement: [self isOpen and: [arrows ~~ #both]] action:  #makeBothArrows.
                 aMenu add: 'customize arrows' translated action: #customizeArrows:.
                 (self hasProperty: #arrowSpec)
                         ifTrue: [aMenu add: 'standard arrows' translated action: #standardArrows]].!

Item was changed:
  ----- Method: PolygonMorph>>dropVertex:event:fromHandle: (in category 'editing') -----
+ dropVertex: ix event: evt fromHandle: handle
+        "Leave vertex in new position. If dropped ontop another vertex delete
+        this one.
- dropVertex: ix event: evt fromHandle: handle
-        "Leave vertex in new position. If dropped ontop another vertex delete this one.
         Check for too few vertices before deleting. The alternative
+        is not pretty -wiz"
+        | p world |
-                                is not pretty -wiz"
-        | p |
         p := vertices at: ix.
+       
+        "snap the dropped vertex to a vertex of another PolygonMorph if it is in near proximity"
+        self isSnappingToOtherPolygons
+                ifTrue: [world := Project current world.
+                        world submorphs
+                                do: [:each | ((each respondsTo: #vertices)
+                                                        and: [each ~= self])
+                                                ifTrue: [each vertices
+                                                                do: [:otherMorphsVertex | (otherMorphsVertex dist: p)
+                                                                                        < 3
+                                                                                ifTrue: [vertices at: ix put: otherMorphsVertex]]]]].
         (vertices size >= 2
                         and: ["check for too few vertices before deleting. The alternative
                                 is not pretty -wiz"
                                 ((vertices atWrap: ix - 1)
                                                 dist: p)
                                                 < 3
                                         or: [((vertices atWrap: ix + 1)
                                                         dist: p)
                                                         < 3]])
                 ifTrue: ["Drag a vertex onto its neighbor means delete"
+                        self deleteVertexAt: ix].
-                                self deleteVertexAt: ix .].
         evt shiftPressed
                 ifTrue: [self removeHandles]
                 ifFalse: [self addHandles
                         "remove then add to recreate"]!

Item was changed:
  ----- Method: PolygonMorph>>initialize (in category 'initialization') -----
  initialize
  "initialize the state of the receiver"
         super initialize.
  ""
         vertices := Array
                                 with: 5 @ 0
                                 with: 20 @ 10
                                 with: 0 @ 20.
         closed := true.
         smoothCurve := false.
         arrows := #none.
+        snapToOtherPolygons := false.
         self computeBounds!

Item was added:
+ ----- Method: PolygonMorph>>isSnappingToOtherPolygons (in category 'access') -----
+ isSnappingToOtherPolygons
+   ^snapToOtherPolygons ifNil:[ snapToOtherPolygons := false].
+  !

Item was added:
+ ----- Method: PolygonMorph>>toggleSnapToOtherPolygons (in category 'menu') -----
+ toggleSnapToOtherPolygons
+      ^snapToOtherPolygons := snapToOtherPolygons not!




Carpe Squeak!
Reply | Threaded
Open this post in threaded view
|

Re: The Inbox: Morphic-kfr.1620.mcz

Karl Ramberg
Hi,
Thanks for checking.
The method had gone into the wrong category so it was in wrong package (Etoys). :-(
I'll move the method to the right category and reupload

Best,
Karl

On Tue, Feb 4, 2020 at 6:34 PM Thiede, Christoph <[hidden email]> wrote:

Nice idea :-)


But when I merge this commit, it misses #snapToOtherPolygonPhrase:




Btw, #snapToOtherPolyongs itself is also absent in my image. Are you depending on another inbox commit?


Best,

Christoph



Von: Squeak-dev <[hidden email]> im Auftrag von [hidden email] <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 17:56:00
An: [hidden email]
Betreff: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
A new version of Morphic was added to project The Inbox:
http://source.squeak.org/inbox/Morphic-kfr.1620.mcz

==================== Summary ====================

Name: Morphic-kfr.1620
Author: kfr
Time: 4 February 2020, 5:55:31.268512 pm
UUID: 741c35fd-d7ac-e54b-9199-ed1dda9e4df9
Ancestors: Morphic-kfr.1619

Enhancement for PolygonMorph. When a vertex is dropped it will snap if it is close to another PolygonMorphs vertex. It is possible to toggle functionality on/ off in menu

=============== Diff against Morphic-kfr.1619 ===============

Item was changed:
  BorderedMorph subclass: #PolygonMorph
+        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm snapToOtherPolygons'
-        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm'
         classVariableNames: ''
         poolDictionaries: ''
         category: 'Morphic-Basic'!
 
  !PolygonMorph commentStamp: 'md 2/24/2006 20:34' prior: 0!
  This class implements a morph which can behave as four different objects depending on the the following two facts:
  - is it OPEN or CLOSED?
  - is it SEGMENTED or SMOOTHED.
 
  1. The OPEN and SEGMENTED variant looks like polyline.
 
  2. The OPEN and SMOOTHED variant looks like spline (kind of curve)
 
  3. The CLOSED and SEGMENTED variant looks like polygon. This is actually what you get when you do
         PolygonMorph new openInWorld
  You get a triangle. See below how to manipulate these objects...
 
  4. The CLOSED and SMOOTHED variant looks like blob (???)
 
  Prototypes of this morph can also be found in "Object Catalog". Several (different variants) of this object are among "Basic" morphs.
 
  Explore the assiciated morph-menu. It enables you
  - to toggle showing of "handles". They make it possible to
         - reposition already existing vertices (by moving yellow handles)
         - create new vertices (by moving green handles)
         - delete already existing vertices (by dragging and dropping one yellow handle closely
           nearby the adjacent yellow handle
    Handles can be made visible/hidden by shift+leftclicking the morph. This way it is possible
    to quickly show handles, adjust vertices and then again hide handles.
  - making closed polygon open, i.e. converting it to a curve (and vice versa)
  - toggle smoothed/segmented line/outline
  - set up custom dashing (for line, curves or borders of closed polygons
  - set up custom arrow-heads (for lines resp. curves)
 
  ------------------------------------------------------------------------------------------
  Implementation notes:
 
  This class combines the old Polygon and Curve classes.
 
  The 1-bit fillForm to make display and containment tests reasonably fast.  However, this functionality is in the process of being supplanted by balloon capabilities, which should eventually provide anti-aliasing as well.
 
  wiz 7/18/2004 21:26
  s have made some changes to this class to
 
  1) correct some bugs associated with one vertex polygons.
 
  2) prepare for some enhancements with new curves.
 
  3) add shaping items to menu.!

Item was changed:
  ----- Method: PolygonMorph>>addCustomMenuItems:hand: (in category 'menu') -----
  addCustomMenuItems: aMenu hand: aHandMorph
         "Add morph-specific items to the given menu which was invoked by the given hand.  This method provides is invoked both from the halo-menu and from the control-menu regimes."
 
         super addCustomMenuItems: aMenu hand: aHandMorph.
         aMenu addUpdating: #handlesShowingPhrase target: self action: #showOrHideHandles.
         vertices size > 2 ifTrue:
                 [aMenu addUpdating: #openOrClosePhrase target: self action: #makeOpenOrClosed].
 
         aMenu addUpdating: #smoothPhrase target: self action: #toggleSmoothing.
+        aMenu addUpdating: #snapToOtherPolygonPhrase target: self action: #toggleSnapToOtherPolygons.
         aMenu addLine.
         aMenu add: 'specify dashed line' translated action:  #specifyDashedLine.
 
         self isOpen ifTrue:
                 [aMenu addLine.
                 aMenu addWithLabel: '---' enablement: [self isOpen and: [arrows ~~ #none]] action:  #makeNoArrows.
                 aMenu addWithLabel: '-->' enablement: [self isOpen and: [arrows ~~ #forward]] action:  #makeForwardArrow.
                 aMenu addWithLabel: '<--' enablement: [self isOpen and: [arrows ~~ #back]] action:  #makeBackArrow.
                 aMenu addWithLabel: '<->' enablement: [self isOpen and: [arrows ~~ #both]] action:  #makeBothArrows.
                 aMenu add: 'customize arrows' translated action: #customizeArrows:.
                 (self hasProperty: #arrowSpec)
                         ifTrue: [aMenu add: 'standard arrows' translated action: #standardArrows]].!

Item was changed:
  ----- Method: PolygonMorph>>dropVertex:event:fromHandle: (in category 'editing') -----
+ dropVertex: ix event: evt fromHandle: handle
+        "Leave vertex in new position. If dropped ontop another vertex delete
+        this one.
- dropVertex: ix event: evt fromHandle: handle
-        "Leave vertex in new position. If dropped ontop another vertex delete this one.
         Check for too few vertices before deleting. The alternative
+        is not pretty -wiz"
+        | p world |
-                                is not pretty -wiz"
-        | p |
         p := vertices at: ix.
+       
+        "snap the dropped vertex to a vertex of another PolygonMorph if it is in near proximity"
+        self isSnappingToOtherPolygons
+                ifTrue: [world := Project current world.
+                        world submorphs
+                                do: [:each | ((each respondsTo: #vertices)
+                                                        and: [each ~= self])
+                                                ifTrue: [each vertices
+                                                                do: [:otherMorphsVertex | (otherMorphsVertex dist: p)
+                                                                                        < 3
+                                                                                ifTrue: [vertices at: ix put: otherMorphsVertex]]]]].
         (vertices size >= 2
                         and: ["check for too few vertices before deleting. The alternative
                                 is not pretty -wiz"
                                 ((vertices atWrap: ix - 1)
                                                 dist: p)
                                                 < 3
                                         or: [((vertices atWrap: ix + 1)
                                                         dist: p)
                                                         < 3]])
                 ifTrue: ["Drag a vertex onto its neighbor means delete"
+                        self deleteVertexAt: ix].
-                                self deleteVertexAt: ix .].
         evt shiftPressed
                 ifTrue: [self removeHandles]
                 ifFalse: [self addHandles
                         "remove then add to recreate"]!

Item was changed:
  ----- Method: PolygonMorph>>initialize (in category 'initialization') -----
  initialize
  "initialize the state of the receiver"
         super initialize.
  ""
         vertices := Array
                                 with: 5 @ 0
                                 with: 20 @ 10
                                 with: 0 @ 20.
         closed := true.
         smoothCurve := false.
         arrows := #none.
+        snapToOtherPolygons := false.
         self computeBounds!

Item was added:
+ ----- Method: PolygonMorph>>isSnappingToOtherPolygons (in category 'access') -----
+ isSnappingToOtherPolygons
+   ^snapToOtherPolygons ifNil:[ snapToOtherPolygons := false].
+  !

Item was added:
+ ----- Method: PolygonMorph>>toggleSnapToOtherPolygons (in category 'menu') -----
+ toggleSnapToOtherPolygons
+      ^snapToOtherPolygons := snapToOtherPolygons not!





Reply | Threaded
Open this post in threaded view
|

Re: The Inbox: Morphic-kfr.1620.mcz

Christoph Thiede

Wow, I did not know that you can overwrite an existing commit in the inbox :o


Unfortunately, I can't reproduce the feature. How do I use the snapping after activating it in the menu? Probably the threshold is too small for a visible effect on a 250 % DPI screen.


Best,
Christoph

Von: Squeak-dev <[hidden email]> im Auftrag von karl ramberg <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 19:55:12
An: The general-purpose Squeak developers list
Betreff: Re: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
Hi,
Thanks for checking.
The method had gone into the wrong category so it was in wrong package (Etoys). :-(
I'll move the method to the right category and reupload

Best,
Karl

On Tue, Feb 4, 2020 at 6:34 PM Thiede, Christoph <[hidden email]> wrote:

Nice idea :-)


But when I merge this commit, it misses #snapToOtherPolygonPhrase:




Btw, #snapToOtherPolyongs itself is also absent in my image. Are you depending on another inbox commit?


Best,

Christoph



Von: Squeak-dev <[hidden email]> im Auftrag von [hidden email] <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 17:56:00
An: [hidden email]
Betreff: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
A new version of Morphic was added to project The Inbox:
http://source.squeak.org/inbox/Morphic-kfr.1620.mcz

==================== Summary ====================

Name: Morphic-kfr.1620
Author: kfr
Time: 4 February 2020, 5:55:31.268512 pm
UUID: 741c35fd-d7ac-e54b-9199-ed1dda9e4df9
Ancestors: Morphic-kfr.1619

Enhancement for PolygonMorph. When a vertex is dropped it will snap if it is close to another PolygonMorphs vertex. It is possible to toggle functionality on/ off in menu

=============== Diff against Morphic-kfr.1619 ===============

Item was changed:
  BorderedMorph subclass: #PolygonMorph
+        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm snapToOtherPolygons'
-        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm'
         classVariableNames: ''
         poolDictionaries: ''
         category: 'Morphic-Basic'!
 
  !PolygonMorph commentStamp: 'md 2/24/2006 20:34' prior: 0!
  This class implements a morph which can behave as four different objects depending on the the following two facts:
  - is it OPEN or CLOSED?
  - is it SEGMENTED or SMOOTHED.
 
  1. The OPEN and SEGMENTED variant looks like polyline.
 
  2. The OPEN and SMOOTHED variant looks like spline (kind of curve)
 
  3. The CLOSED and SEGMENTED variant looks like polygon. This is actually what you get when you do
         PolygonMorph new openInWorld
  You get a triangle. See below how to manipulate these objects...
 
  4. The CLOSED and SMOOTHED variant looks like blob (???)
 
  Prototypes of this morph can also be found in "Object Catalog". Several (different variants) of this object are among "Basic" morphs.
 
  Explore the assiciated morph-menu. It enables you
  - to toggle showing of "handles". They make it possible to
         - reposition already existing vertices (by moving yellow handles)
         - create new vertices (by moving green handles)
         - delete already existing vertices (by dragging and dropping one yellow handle closely
           nearby the adjacent yellow handle
    Handles can be made visible/hidden by shift+leftclicking the morph. This way it is possible
    to quickly show handles, adjust vertices and then again hide handles.
  - making closed polygon open, i.e. converting it to a curve (and vice versa)
  - toggle smoothed/segmented line/outline
  - set up custom dashing (for line, curves or borders of closed polygons
  - set up custom arrow-heads (for lines resp. curves)
 
  ------------------------------------------------------------------------------------------
  Implementation notes:
 
  This class combines the old Polygon and Curve classes.
 
  The 1-bit fillForm to make display and containment tests reasonably fast.  However, this functionality is in the process of being supplanted by balloon capabilities, which should eventually provide anti-aliasing as well.
 
  wiz 7/18/2004 21:26
  s have made some changes to this class to
 
  1) correct some bugs associated with one vertex polygons.
 
  2) prepare for some enhancements with new curves.
 
  3) add shaping items to menu.!

Item was changed:
  ----- Method: PolygonMorph>>addCustomMenuItems:hand: (in category 'menu') -----
  addCustomMenuItems: aMenu hand: aHandMorph
         "Add morph-specific items to the given menu which was invoked by the given hand.  This method provides is invoked both from the halo-menu and from the control-menu regimes."
 
         super addCustomMenuItems: aMenu hand: aHandMorph.
         aMenu addUpdating: #handlesShowingPhrase target: self action: #showOrHideHandles.
         vertices size > 2 ifTrue:
                 [aMenu addUpdating: #openOrClosePhrase target: self action: #makeOpenOrClosed].
 
         aMenu addUpdating: #smoothPhrase target: self action: #toggleSmoothing.
+        aMenu addUpdating: #snapToOtherPolygonPhrase target: self action: #toggleSnapToOtherPolygons.
         aMenu addLine.
         aMenu add: 'specify dashed line' translated action:  #specifyDashedLine.
 
         self isOpen ifTrue:
                 [aMenu addLine.
                 aMenu addWithLabel: '---' enablement: [self isOpen and: [arrows ~~ #none]] action:  #makeNoArrows.
                 aMenu addWithLabel: '-->' enablement: [self isOpen and: [arrows ~~ #forward]] action:  #makeForwardArrow.
                 aMenu addWithLabel: '<--' enablement: [self isOpen and: [arrows ~~ #back]] action:  #makeBackArrow.
                 aMenu addWithLabel: '<->' enablement: [self isOpen and: [arrows ~~ #both]] action:  #makeBothArrows.
                 aMenu add: 'customize arrows' translated action: #customizeArrows:.
                 (self hasProperty: #arrowSpec)
                         ifTrue: [aMenu add: 'standard arrows' translated action: #standardArrows]].!

Item was changed:
  ----- Method: PolygonMorph>>dropVertex:event:fromHandle: (in category 'editing') -----
+ dropVertex: ix event: evt fromHandle: handle
+        "Leave vertex in new position. If dropped ontop another vertex delete
+        this one.
- dropVertex: ix event: evt fromHandle: handle
-        "Leave vertex in new position. If dropped ontop another vertex delete this one.
         Check for too few vertices before deleting. The alternative
+        is not pretty -wiz"
+        | p world |
-                                is not pretty -wiz"
-        | p |
         p := vertices at: ix.
+       
+        "snap the dropped vertex to a vertex of another PolygonMorph if it is in near proximity"
+        self isSnappingToOtherPolygons
+                ifTrue: [world := Project current world.
+                        world submorphs
+                                do: [:each | ((each respondsTo: #vertices)
+                                                        and: [each ~= self])
+                                                ifTrue: [each vertices
+                                                                do: [:otherMorphsVertex | (otherMorphsVertex dist: p)
+                                                                                        < 3
+                                                                                ifTrue: [vertices at: ix put: otherMorphsVertex]]]]].
         (vertices size >= 2
                         and: ["check for too few vertices before deleting. The alternative
                                 is not pretty -wiz"
                                 ((vertices atWrap: ix - 1)
                                                 dist: p)
                                                 < 3
                                         or: [((vertices atWrap: ix + 1)
                                                         dist: p)
                                                         < 3]])
                 ifTrue: ["Drag a vertex onto its neighbor means delete"
+                        self deleteVertexAt: ix].
-                                self deleteVertexAt: ix .].
         evt shiftPressed
                 ifTrue: [self removeHandles]
                 ifFalse: [self addHandles
                         "remove then add to recreate"]!

Item was changed:
  ----- Method: PolygonMorph>>initialize (in category 'initialization') -----
  initialize
  "initialize the state of the receiver"
         super initialize.
  ""
         vertices := Array
                                 with: 5 @ 0
                                 with: 20 @ 10
                                 with: 0 @ 20.
         closed := true.
         smoothCurve := false.
         arrows := #none.
+        snapToOtherPolygons := false.
         self computeBounds!

Item was added:
+ ----- Method: PolygonMorph>>isSnappingToOtherPolygons (in category 'access') -----
+ isSnappingToOtherPolygons
+   ^snapToOtherPolygons ifNil:[ snapToOtherPolygons := false].
+  !

Item was added:
+ ----- Method: PolygonMorph>>toggleSnapToOtherPolygons (in category 'menu') -----
+ toggleSnapToOtherPolygons
+      ^snapToOtherPolygons := snapToOtherPolygons not!





Carpe Squeak!
Reply | Threaded
Open this post in threaded view
|

Re: The Inbox: Morphic-kfr.1620.mcz

Karl Ramberg
Hi
I moved the previous commit to treated.

Threshold is same as for deleting by dropping vertices on top of each other:  3
Maybe it should be bigger to be convenient, you can test changing the value here:

PolygonMorph>>dropVertex:event:fromHandle
...
    (otherMorphsVertex dist: p) < 3
            ifTrue: [vertices at: ix put: otherMorphsVertex]


Best,
Karl

On Tue, Feb 4, 2020 at 8:05 PM Thiede, Christoph <[hidden email]> wrote:

Wow, I did not know that you can overwrite an existing commit in the inbox :o


Unfortunately, I can't reproduce the feature. How do I use the snapping after activating it in the menu? Probably the threshold is too small for a visible effect on a 250 % DPI screen.


Best,
Christoph

Von: Squeak-dev <[hidden email]> im Auftrag von karl ramberg <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 19:55:12
An: The general-purpose Squeak developers list
Betreff: Re: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
Hi,
Thanks for checking.
The method had gone into the wrong category so it was in wrong package (Etoys). :-(
I'll move the method to the right category and reupload

Best,
Karl

On Tue, Feb 4, 2020 at 6:34 PM Thiede, Christoph <[hidden email]> wrote:

Nice idea :-)


But when I merge this commit, it misses #snapToOtherPolygonPhrase:




Btw, #snapToOtherPolyongs itself is also absent in my image. Are you depending on another inbox commit?


Best,

Christoph



Von: Squeak-dev <[hidden email]> im Auftrag von [hidden email] <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 17:56:00
An: [hidden email]
Betreff: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
A new version of Morphic was added to project The Inbox:
http://source.squeak.org/inbox/Morphic-kfr.1620.mcz

==================== Summary ====================

Name: Morphic-kfr.1620
Author: kfr
Time: 4 February 2020, 5:55:31.268512 pm
UUID: 741c35fd-d7ac-e54b-9199-ed1dda9e4df9
Ancestors: Morphic-kfr.1619

Enhancement for PolygonMorph. When a vertex is dropped it will snap if it is close to another PolygonMorphs vertex. It is possible to toggle functionality on/ off in menu

=============== Diff against Morphic-kfr.1619 ===============

Item was changed:
  BorderedMorph subclass: #PolygonMorph
+        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm snapToOtherPolygons'
-        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm'
         classVariableNames: ''
         poolDictionaries: ''
         category: 'Morphic-Basic'!
 
  !PolygonMorph commentStamp: 'md 2/24/2006 20:34' prior: 0!
  This class implements a morph which can behave as four different objects depending on the the following two facts:
  - is it OPEN or CLOSED?
  - is it SEGMENTED or SMOOTHED.
 
  1. The OPEN and SEGMENTED variant looks like polyline.
 
  2. The OPEN and SMOOTHED variant looks like spline (kind of curve)
 
  3. The CLOSED and SEGMENTED variant looks like polygon. This is actually what you get when you do
         PolygonMorph new openInWorld
  You get a triangle. See below how to manipulate these objects...
 
  4. The CLOSED and SMOOTHED variant looks like blob (???)
 
  Prototypes of this morph can also be found in "Object Catalog". Several (different variants) of this object are among "Basic" morphs.
 
  Explore the assiciated morph-menu. It enables you
  - to toggle showing of "handles". They make it possible to
         - reposition already existing vertices (by moving yellow handles)
         - create new vertices (by moving green handles)
         - delete already existing vertices (by dragging and dropping one yellow handle closely
           nearby the adjacent yellow handle
    Handles can be made visible/hidden by shift+leftclicking the morph. This way it is possible
    to quickly show handles, adjust vertices and then again hide handles.
  - making closed polygon open, i.e. converting it to a curve (and vice versa)
  - toggle smoothed/segmented line/outline
  - set up custom dashing (for line, curves or borders of closed polygons
  - set up custom arrow-heads (for lines resp. curves)
 
  ------------------------------------------------------------------------------------------
  Implementation notes:
 
  This class combines the old Polygon and Curve classes.
 
  The 1-bit fillForm to make display and containment tests reasonably fast.  However, this functionality is in the process of being supplanted by balloon capabilities, which should eventually provide anti-aliasing as well.
 
  wiz 7/18/2004 21:26
  s have made some changes to this class to
 
  1) correct some bugs associated with one vertex polygons.
 
  2) prepare for some enhancements with new curves.
 
  3) add shaping items to menu.!

Item was changed:
  ----- Method: PolygonMorph>>addCustomMenuItems:hand: (in category 'menu') -----
  addCustomMenuItems: aMenu hand: aHandMorph
         "Add morph-specific items to the given menu which was invoked by the given hand.  This method provides is invoked both from the halo-menu and from the control-menu regimes."
 
         super addCustomMenuItems: aMenu hand: aHandMorph.
         aMenu addUpdating: #handlesShowingPhrase target: self action: #showOrHideHandles.
         vertices size > 2 ifTrue:
                 [aMenu addUpdating: #openOrClosePhrase target: self action: #makeOpenOrClosed].
 
         aMenu addUpdating: #smoothPhrase target: self action: #toggleSmoothing.
+        aMenu addUpdating: #snapToOtherPolygonPhrase target: self action: #toggleSnapToOtherPolygons.
         aMenu addLine.
         aMenu add: 'specify dashed line' translated action:  #specifyDashedLine.
 
         self isOpen ifTrue:
                 [aMenu addLine.
                 aMenu addWithLabel: '---' enablement: [self isOpen and: [arrows ~~ #none]] action:  #makeNoArrows.
                 aMenu addWithLabel: '-->' enablement: [self isOpen and: [arrows ~~ #forward]] action:  #makeForwardArrow.
                 aMenu addWithLabel: '<--' enablement: [self isOpen and: [arrows ~~ #back]] action:  #makeBackArrow.
                 aMenu addWithLabel: '<->' enablement: [self isOpen and: [arrows ~~ #both]] action:  #makeBothArrows.
                 aMenu add: 'customize arrows' translated action: #customizeArrows:.
                 (self hasProperty: #arrowSpec)
                         ifTrue: [aMenu add: 'standard arrows' translated action: #standardArrows]].!

Item was changed:
  ----- Method: PolygonMorph>>dropVertex:event:fromHandle: (in category 'editing') -----
+ dropVertex: ix event: evt fromHandle: handle
+        "Leave vertex in new position. If dropped ontop another vertex delete
+        this one.
- dropVertex: ix event: evt fromHandle: handle
-        "Leave vertex in new position. If dropped ontop another vertex delete this one.
         Check for too few vertices before deleting. The alternative
+        is not pretty -wiz"
+        | p world |
-                                is not pretty -wiz"
-        | p |
         p := vertices at: ix.
+       
+        "snap the dropped vertex to a vertex of another PolygonMorph if it is in near proximity"
+        self isSnappingToOtherPolygons
+                ifTrue: [world := Project current world.
+                        world submorphs
+                                do: [:each | ((each respondsTo: #vertices)
+                                                        and: [each ~= self])
+                                                ifTrue: [each vertices
+                                                                do: [:otherMorphsVertex | (otherMorphsVertex dist: p)
+                                                                                        < 3
+                                                                                ifTrue: [vertices at: ix put: otherMorphsVertex]]]]].
         (vertices size >= 2
                         and: ["check for too few vertices before deleting. The alternative
                                 is not pretty -wiz"
                                 ((vertices atWrap: ix - 1)
                                                 dist: p)
                                                 < 3
                                         or: [((vertices atWrap: ix + 1)
                                                         dist: p)
                                                         < 3]])
                 ifTrue: ["Drag a vertex onto its neighbor means delete"
+                        self deleteVertexAt: ix].
-                                self deleteVertexAt: ix .].
         evt shiftPressed
                 ifTrue: [self removeHandles]
                 ifFalse: [self addHandles
                         "remove then add to recreate"]!

Item was changed:
  ----- Method: PolygonMorph>>initialize (in category 'initialization') -----
  initialize
  "initialize the state of the receiver"
         super initialize.
  ""
         vertices := Array
                                 with: 5 @ 0
                                 with: 20 @ 10
                                 with: 0 @ 20.
         closed := true.
         smoothCurve := false.
         arrows := #none.
+        snapToOtherPolygons := false.
         self computeBounds!

Item was added:
+ ----- Method: PolygonMorph>>isSnappingToOtherPolygons (in category 'access') -----
+ isSnappingToOtherPolygons
+   ^snapToOtherPolygons ifNil:[ snapToOtherPolygons := false].
+  !

Item was added:
+ ----- Method: PolygonMorph>>toggleSnapToOtherPolygons (in category 'menu') -----
+ toggleSnapToOtherPolygons
+      ^snapToOtherPolygons := snapToOtherPolygons not!






Reply | Threaded
Open this post in threaded view
|

Re: The Inbox: Morphic-kfr.1620.mcz

Christoph Thiede

Hi Karl,


thanks for the tip. It would be nice to make an accessor #dragThreshold for it and compute it by 3 * RealEstateAgent scaleFactor, for example. What do you think about it? :-)


Best,

Christoph



Von: Squeak-dev <[hidden email]> im Auftrag von karl ramberg <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 20:25 Uhr
An: The general-purpose Squeak developers list
Betreff: Re: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
Hi
I moved the previous commit to treated.

Threshold is same as for deleting by dropping vertices on top of each other:  3
Maybe it should be bigger to be convenient, you can test changing the value here:

PolygonMorph>>dropVertex:event:fromHandle
...
    (otherMorphsVertex dist: p) < 3
            ifTrue: [vertices at: ix put: otherMorphsVertex]


Best,
Karl

On Tue, Feb 4, 2020 at 8:05 PM Thiede, Christoph <[hidden email]> wrote:

Wow, I did not know that you can overwrite an existing commit in the inbox :o


Unfortunately, I can't reproduce the feature. How do I use the snapping after activating it in the menu? Probably the threshold is too small for a visible effect on a 250 % DPI screen.


Best,
Christoph

Von: Squeak-dev <[hidden email]> im Auftrag von karl ramberg <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 19:55:12
An: The general-purpose Squeak developers list
Betreff: Re: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
Hi,
Thanks for checking.
The method had gone into the wrong category so it was in wrong package (Etoys). :-(
I'll move the method to the right category and reupload

Best,
Karl

On Tue, Feb 4, 2020 at 6:34 PM Thiede, Christoph <[hidden email]> wrote:

Nice idea :-)


But when I merge this commit, it misses #snapToOtherPolygonPhrase:




Btw, #snapToOtherPolyongs itself is also absent in my image. Are you depending on another inbox commit?


Best,

Christoph



Von: Squeak-dev <[hidden email]> im Auftrag von [hidden email] <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 17:56:00
An: [hidden email]
Betreff: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
A new version of Morphic was added to project The Inbox:
http://source.squeak.org/inbox/Morphic-kfr.1620.mcz

==================== Summary ====================

Name: Morphic-kfr.1620
Author: kfr
Time: 4 February 2020, 5:55:31.268512 pm
UUID: 741c35fd-d7ac-e54b-9199-ed1dda9e4df9
Ancestors: Morphic-kfr.1619

Enhancement for PolygonMorph. When a vertex is dropped it will snap if it is close to another PolygonMorphs vertex. It is possible to toggle functionality on/ off in menu

=============== Diff against Morphic-kfr.1619 ===============

Item was changed:
  BorderedMorph subclass: #PolygonMorph
+        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm snapToOtherPolygons'
-        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm'
         classVariableNames: ''
         poolDictionaries: ''
         category: 'Morphic-Basic'!
 
  !PolygonMorph commentStamp: 'md 2/24/2006 20:34' prior: 0!
  This class implements a morph which can behave as four different objects depending on the the following two facts:
  - is it OPEN or CLOSED?
  - is it SEGMENTED or SMOOTHED.
 
  1. The OPEN and SEGMENTED variant looks like polyline.
 
  2. The OPEN and SMOOTHED variant looks like spline (kind of curve)
 
  3. The CLOSED and SEGMENTED variant looks like polygon. This is actually what you get when you do
         PolygonMorph new openInWorld
  You get a triangle. See below how to manipulate these objects...
 
  4. The CLOSED and SMOOTHED variant looks like blob (???)
 
  Prototypes of this morph can also be found in "Object Catalog". Several (different variants) of this object are among "Basic" morphs.
 
  Explore the assiciated morph-menu. It enables you
  - to toggle showing of "handles". They make it possible to
         - reposition already existing vertices (by moving yellow handles)
         - create new vertices (by moving green handles)
         - delete already existing vertices (by dragging and dropping one yellow handle closely
           nearby the adjacent yellow handle
    Handles can be made visible/hidden by shift+leftclicking the morph. This way it is possible
    to quickly show handles, adjust vertices and then again hide handles.
  - making closed polygon open, i.e. converting it to a curve (and vice versa)
  - toggle smoothed/segmented line/outline
  - set up custom dashing (for line, curves or borders of closed polygons
  - set up custom arrow-heads (for lines resp. curves)
 
  ------------------------------------------------------------------------------------------
  Implementation notes:
 
  This class combines the old Polygon and Curve classes.
 
  The 1-bit fillForm to make display and containment tests reasonably fast.  However, this functionality is in the process of being supplanted by balloon capabilities, which should eventually provide anti-aliasing as well.
 
  wiz 7/18/2004 21:26
  s have made some changes to this class to
 
  1) correct some bugs associated with one vertex polygons.
 
  2) prepare for some enhancements with new curves.
 
  3) add shaping items to menu.!

Item was changed:
  ----- Method: PolygonMorph>>addCustomMenuItems:hand: (in category 'menu') -----
  addCustomMenuItems: aMenu hand: aHandMorph
         "Add morph-specific items to the given menu which was invoked by the given hand.  This method provides is invoked both from the halo-menu and from the control-menu regimes."
 
         super addCustomMenuItems: aMenu hand: aHandMorph.
         aMenu addUpdating: #handlesShowingPhrase target: self action: #showOrHideHandles.
         vertices size > 2 ifTrue:
                 [aMenu addUpdating: #openOrClosePhrase target: self action: #makeOpenOrClosed].
 
         aMenu addUpdating: #smoothPhrase target: self action: #toggleSmoothing.
+        aMenu addUpdating: #snapToOtherPolygonPhrase target: self action: #toggleSnapToOtherPolygons.
         aMenu addLine.
         aMenu add: 'specify dashed line' translated action:  #specifyDashedLine.
 
         self isOpen ifTrue:
                 [aMenu addLine.
                 aMenu addWithLabel: '---' enablement: [self isOpen and: [arrows ~~ #none]] action:  #makeNoArrows.
                 aMenu addWithLabel: '-->' enablement: [self isOpen and: [arrows ~~ #forward]] action:  #makeForwardArrow.
                 aMenu addWithLabel: '<--' enablement: [self isOpen and: [arrows ~~ #back]] action:  #makeBackArrow.
                 aMenu addWithLabel: '<->' enablement: [self isOpen and: [arrows ~~ #both]] action:  #makeBothArrows.
                 aMenu add: 'customize arrows' translated action: #customizeArrows:.
                 (self hasProperty: #arrowSpec)
                         ifTrue: [aMenu add: 'standard arrows' translated action: #standardArrows]].!

Item was changed:
  ----- Method: PolygonMorph>>dropVertex:event:fromHandle: (in category 'editing') -----
+ dropVertex: ix event: evt fromHandle: handle
+        "Leave vertex in new position. If dropped ontop another vertex delete
+        this one.
- dropVertex: ix event: evt fromHandle: handle
-        "Leave vertex in new position. If dropped ontop another vertex delete this one.
         Check for too few vertices before deleting. The alternative
+        is not pretty -wiz"
+        | p world |
-                                is not pretty -wiz"
-        | p |
         p := vertices at: ix.
+       
+        "snap the dropped vertex to a vertex of another PolygonMorph if it is in near proximity"
+        self isSnappingToOtherPolygons
+                ifTrue: [world := Project current world.
+                        world submorphs
+                                do: [:each | ((each respondsTo: #vertices)
+                                                        and: [each ~= self])
+                                                ifTrue: [each vertices
+                                                                do: [:otherMorphsVertex | (otherMorphsVertex dist: p)
+                                                                                        < 3
+                                                                                ifTrue: [vertices at: ix put: otherMorphsVertex]]]]].
         (vertices size >= 2
                         and: ["check for too few vertices before deleting. The alternative
                                 is not pretty -wiz"
                                 ((vertices atWrap: ix - 1)
                                                 dist: p)
                                                 < 3
                                         or: [((vertices atWrap: ix + 1)
                                                         dist: p)
                                                         < 3]])
                 ifTrue: ["Drag a vertex onto its neighbor means delete"
+                        self deleteVertexAt: ix].
-                                self deleteVertexAt: ix .].
         evt shiftPressed
                 ifTrue: [self removeHandles]
                 ifFalse: [self addHandles
                         "remove then add to recreate"]!

Item was changed:
  ----- Method: PolygonMorph>>initialize (in category 'initialization') -----
  initialize
  "initialize the state of the receiver"
         super initialize.
  ""
         vertices := Array
                                 with: 5 @ 0
                                 with: 20 @ 10
                                 with: 0 @ 20.
         closed := true.
         smoothCurve := false.
         arrows := #none.
+        snapToOtherPolygons := false.
         self computeBounds!

Item was added:
+ ----- Method: PolygonMorph>>isSnappingToOtherPolygons (in category 'access') -----
+ isSnappingToOtherPolygons
+   ^snapToOtherPolygons ifNil:[ snapToOtherPolygons := false].
+  !

Item was added:
+ ----- Method: PolygonMorph>>toggleSnapToOtherPolygons (in category 'menu') -----
+ toggleSnapToOtherPolygons
+      ^snapToOtherPolygons := snapToOtherPolygons not!






Carpe Squeak!
Reply | Threaded
Open this post in threaded view
|

Re: The Inbox: Morphic-kfr.1620.mcz

Karl Ramberg
Hi,
That sounds like a good idea.

It could also be nice if the snapping had a a keyboard short cut instead of a menu toggle, so it would snap if for example shift was pressed,
But the whole keyboard short cut functionality is quite messy and there is so may layers  I'm not sure if it is worth diving into...

Best,
Karl

On Wed, 5 Feb 2020 at 10:07, Thiede, Christoph <[hidden email]> wrote:

Hi Karl,


thanks for the tip. It would be nice to make an accessor #dragThreshold for it and compute it by 3 * RealEstateAgent scaleFactor, for example. What do you think about it? :-)


Best,

Christoph



Von: Squeak-dev <[hidden email]> im Auftrag von karl ramberg <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 20:25 Uhr

An: The general-purpose Squeak developers list
Betreff: Re: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
Hi
I moved the previous commit to treated.

Threshold is same as for deleting by dropping vertices on top of each other:  3
Maybe it should be bigger to be convenient, you can test changing the value here:

PolygonMorph>>dropVertex:event:fromHandle
...
    (otherMorphsVertex dist: p) < 3
            ifTrue: [vertices at: ix put: otherMorphsVertex]


Best,
Karl

On Tue, Feb 4, 2020 at 8:05 PM Thiede, Christoph <[hidden email]> wrote:

Wow, I did not know that you can overwrite an existing commit in the inbox :o


Unfortunately, I can't reproduce the feature. How do I use the snapping after activating it in the menu? Probably the threshold is too small for a visible effect on a 250 % DPI screen.


Best,
Christoph

Von: Squeak-dev <[hidden email]> im Auftrag von karl ramberg <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 19:55:12
An: The general-purpose Squeak developers list
Betreff: Re: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
Hi,
Thanks for checking.
The method had gone into the wrong category so it was in wrong package (Etoys). :-(
I'll move the method to the right category and reupload

Best,
Karl

On Tue, Feb 4, 2020 at 6:34 PM Thiede, Christoph <[hidden email]> wrote:

Nice idea :-)


But when I merge this commit, it misses #snapToOtherPolygonPhrase:




Btw, #snapToOtherPolyongs itself is also absent in my image. Are you depending on another inbox commit?


Best,

Christoph



Von: Squeak-dev <[hidden email]> im Auftrag von [hidden email] <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 17:56:00
An: [hidden email]
Betreff: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
A new version of Morphic was added to project The Inbox:
http://source.squeak.org/inbox/Morphic-kfr.1620.mcz

==================== Summary ====================

Name: Morphic-kfr.1620
Author: kfr
Time: 4 February 2020, 5:55:31.268512 pm
UUID: 741c35fd-d7ac-e54b-9199-ed1dda9e4df9
Ancestors: Morphic-kfr.1619

Enhancement for PolygonMorph. When a vertex is dropped it will snap if it is close to another PolygonMorphs vertex. It is possible to toggle functionality on/ off in menu

=============== Diff against Morphic-kfr.1619 ===============

Item was changed:
  BorderedMorph subclass: #PolygonMorph
+        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm snapToOtherPolygons'
-        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm'
         classVariableNames: ''
         poolDictionaries: ''
         category: 'Morphic-Basic'!
 
  !PolygonMorph commentStamp: 'md 2/24/2006 20:34' prior: 0!
  This class implements a morph which can behave as four different objects depending on the the following two facts:
  - is it OPEN or CLOSED?
  - is it SEGMENTED or SMOOTHED.
 
  1. The OPEN and SEGMENTED variant looks like polyline.
 
  2. The OPEN and SMOOTHED variant looks like spline (kind of curve)
 
  3. The CLOSED and SEGMENTED variant looks like polygon. This is actually what you get when you do
         PolygonMorph new openInWorld
  You get a triangle. See below how to manipulate these objects...
 
  4. The CLOSED and SMOOTHED variant looks like blob (???)
 
  Prototypes of this morph can also be found in "Object Catalog". Several (different variants) of this object are among "Basic" morphs.
 
  Explore the assiciated morph-menu. It enables you
  - to toggle showing of "handles". They make it possible to
         - reposition already existing vertices (by moving yellow handles)
         - create new vertices (by moving green handles)
         - delete already existing vertices (by dragging and dropping one yellow handle closely
           nearby the adjacent yellow handle
    Handles can be made visible/hidden by shift+leftclicking the morph. This way it is possible
    to quickly show handles, adjust vertices and then again hide handles.
  - making closed polygon open, i.e. converting it to a curve (and vice versa)
  - toggle smoothed/segmented line/outline
  - set up custom dashing (for line, curves or borders of closed polygons
  - set up custom arrow-heads (for lines resp. curves)
 
  ------------------------------------------------------------------------------------------
  Implementation notes:
 
  This class combines the old Polygon and Curve classes.
 
  The 1-bit fillForm to make display and containment tests reasonably fast.  However, this functionality is in the process of being supplanted by balloon capabilities, which should eventually provide anti-aliasing as well.
 
  wiz 7/18/2004 21:26
  s have made some changes to this class to
 
  1) correct some bugs associated with one vertex polygons.
 
  2) prepare for some enhancements with new curves.
 
  3) add shaping items to menu.!

Item was changed:
  ----- Method: PolygonMorph>>addCustomMenuItems:hand: (in category 'menu') -----
  addCustomMenuItems: aMenu hand: aHandMorph
         "Add morph-specific items to the given menu which was invoked by the given hand.  This method provides is invoked both from the halo-menu and from the control-menu regimes."
 
         super addCustomMenuItems: aMenu hand: aHandMorph.
         aMenu addUpdating: #handlesShowingPhrase target: self action: #showOrHideHandles.
         vertices size > 2 ifTrue:
                 [aMenu addUpdating: #openOrClosePhrase target: self action: #makeOpenOrClosed].
 
         aMenu addUpdating: #smoothPhrase target: self action: #toggleSmoothing.
+        aMenu addUpdating: #snapToOtherPolygonPhrase target: self action: #toggleSnapToOtherPolygons.
         aMenu addLine.
         aMenu add: 'specify dashed line' translated action:  #specifyDashedLine.
 
         self isOpen ifTrue:
                 [aMenu addLine.
                 aMenu addWithLabel: '---' enablement: [self isOpen and: [arrows ~~ #none]] action:  #makeNoArrows.
                 aMenu addWithLabel: '-->' enablement: [self isOpen and: [arrows ~~ #forward]] action:  #makeForwardArrow.
                 aMenu addWithLabel: '<--' enablement: [self isOpen and: [arrows ~~ #back]] action:  #makeBackArrow.
                 aMenu addWithLabel: '<->' enablement: [self isOpen and: [arrows ~~ #both]] action:  #makeBothArrows.
                 aMenu add: 'customize arrows' translated action: #customizeArrows:.
                 (self hasProperty: #arrowSpec)
                         ifTrue: [aMenu add: 'standard arrows' translated action: #standardArrows]].!

Item was changed:
  ----- Method: PolygonMorph>>dropVertex:event:fromHandle: (in category 'editing') -----
+ dropVertex: ix event: evt fromHandle: handle
+        "Leave vertex in new position. If dropped ontop another vertex delete
+        this one.
- dropVertex: ix event: evt fromHandle: handle
-        "Leave vertex in new position. If dropped ontop another vertex delete this one.
         Check for too few vertices before deleting. The alternative
+        is not pretty -wiz"
+        | p world |
-                                is not pretty -wiz"
-        | p |
         p := vertices at: ix.
+       
+        "snap the dropped vertex to a vertex of another PolygonMorph if it is in near proximity"
+        self isSnappingToOtherPolygons
+                ifTrue: [world := Project current world.
+                        world submorphs
+                                do: [:each | ((each respondsTo: #vertices)
+                                                        and: [each ~= self])
+                                                ifTrue: [each vertices
+                                                                do: [:otherMorphsVertex | (otherMorphsVertex dist: p)
+                                                                                        < 3
+                                                                                ifTrue: [vertices at: ix put: otherMorphsVertex]]]]].
         (vertices size >= 2
                         and: ["check for too few vertices before deleting. The alternative
                                 is not pretty -wiz"
                                 ((vertices atWrap: ix - 1)
                                                 dist: p)
                                                 < 3
                                         or: [((vertices atWrap: ix + 1)
                                                         dist: p)
                                                         < 3]])
                 ifTrue: ["Drag a vertex onto its neighbor means delete"
+                        self deleteVertexAt: ix].
-                                self deleteVertexAt: ix .].
         evt shiftPressed
                 ifTrue: [self removeHandles]
                 ifFalse: [self addHandles
                         "remove then add to recreate"]!

Item was changed:
  ----- Method: PolygonMorph>>initialize (in category 'initialization') -----
  initialize
  "initialize the state of the receiver"
         super initialize.
  ""
         vertices := Array
                                 with: 5 @ 0
                                 with: 20 @ 10
                                 with: 0 @ 20.
         closed := true.
         smoothCurve := false.
         arrows := #none.
+        snapToOtherPolygons := false.
         self computeBounds!

Item was added:
+ ----- Method: PolygonMorph>>isSnappingToOtherPolygons (in category 'access') -----
+ isSnappingToOtherPolygons
+   ^snapToOtherPolygons ifNil:[ snapToOtherPolygons := false].
+  !

Item was added:
+ ----- Method: PolygonMorph>>toggleSnapToOtherPolygons (in category 'menu') -----
+ toggleSnapToOtherPolygons
+      ^snapToOtherPolygons := snapToOtherPolygons not!







Reply | Threaded
Open this post in threaded view
|

Re: The Inbox: Morphic-kfr.1620.mcz

Christoph Thiede

Hm, shift may be indeed complicated, because shift + red button often is mapped to yellow, afaik ...

It could be also useful if snapping, if activated, already happened before releasing the mouse button.

Choose what you think is the best :-)

Best,
Christoph

Von: Squeak-dev <[hidden email]> im Auftrag von karl ramberg <[hidden email]>
Gesendet: Mittwoch, 5. Februar 2020 17:49:47
An: The general-purpose Squeak developers list
Betreff: Re: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
Hi,
That sounds like a good idea.

It could also be nice if the snapping had a a keyboard short cut instead of a menu toggle, so it would snap if for example shift was pressed,
But the whole keyboard short cut functionality is quite messy and there is so may layers  I'm not sure if it is worth diving into...

Best,
Karl

On Wed, 5 Feb 2020 at 10:07, Thiede, Christoph <[hidden email]> wrote:

Hi Karl,


thanks for the tip. It would be nice to make an accessor #dragThreshold for it and compute it by 3 * RealEstateAgent scaleFactor, for example. What do you think about it? :-)


Best,

Christoph



Von: Squeak-dev <[hidden email]> im Auftrag von karl ramberg <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 20:25 Uhr

An: The general-purpose Squeak developers list
Betreff: Re: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
Hi
I moved the previous commit to treated.

Threshold is same as for deleting by dropping vertices on top of each other:  3
Maybe it should be bigger to be convenient, you can test changing the value here:

PolygonMorph>>dropVertex:event:fromHandle
...
    (otherMorphsVertex dist: p) < 3
            ifTrue: [vertices at: ix put: otherMorphsVertex]


Best,
Karl

On Tue, Feb 4, 2020 at 8:05 PM Thiede, Christoph <[hidden email]> wrote:

Wow, I did not know that you can overwrite an existing commit in the inbox :o


Unfortunately, I can't reproduce the feature. How do I use the snapping after activating it in the menu? Probably the threshold is too small for a visible effect on a 250 % DPI screen.


Best,
Christoph

Von: Squeak-dev <[hidden email]> im Auftrag von karl ramberg <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 19:55:12
An: The general-purpose Squeak developers list
Betreff: Re: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
Hi,
Thanks for checking.
The method had gone into the wrong category so it was in wrong package (Etoys). :-(
I'll move the method to the right category and reupload

Best,
Karl

On Tue, Feb 4, 2020 at 6:34 PM Thiede, Christoph <[hidden email]> wrote:

Nice idea :-)


But when I merge this commit, it misses #snapToOtherPolygonPhrase:




Btw, #snapToOtherPolyongs itself is also absent in my image. Are you depending on another inbox commit?


Best,

Christoph



Von: Squeak-dev <[hidden email]> im Auftrag von [hidden email] <[hidden email]>
Gesendet: Dienstag, 4. Februar 2020 17:56:00
An: [hidden email]
Betreff: [squeak-dev] The Inbox: Morphic-kfr.1620.mcz
 
A new version of Morphic was added to project The Inbox:
http://source.squeak.org/inbox/Morphic-kfr.1620.mcz

==================== Summary ====================

Name: Morphic-kfr.1620
Author: kfr
Time: 4 February 2020, 5:55:31.268512 pm
UUID: 741c35fd-d7ac-e54b-9199-ed1dda9e4df9
Ancestors: Morphic-kfr.1619

Enhancement for PolygonMorph. When a vertex is dropped it will snap if it is close to another PolygonMorphs vertex. It is possible to toggle functionality on/ off in menu

=============== Diff against Morphic-kfr.1619 ===============

Item was changed:
  BorderedMorph subclass: #PolygonMorph
+        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm snapToOtherPolygons'
-        instanceVariableNames: 'vertices closed filledForm arrows arrowForms smoothCurve curveState borderDashSpec handles borderForm'
         classVariableNames: ''
         poolDictionaries: ''
         category: 'Morphic-Basic'!
 
  !PolygonMorph commentStamp: 'md 2/24/2006 20:34' prior: 0!
  This class implements a morph which can behave as four different objects depending on the the following two facts:
  - is it OPEN or CLOSED?
  - is it SEGMENTED or SMOOTHED.
 
  1. The OPEN and SEGMENTED variant looks like polyline.
 
  2. The OPEN and SMOOTHED variant looks like spline (kind of curve)
 
  3. The CLOSED and SEGMENTED variant looks like polygon. This is actually what you get when you do
         PolygonMorph new openInWorld
  You get a triangle. See below how to manipulate these objects...
 
  4. The CLOSED and SMOOTHED variant looks like blob (???)
 
  Prototypes of this morph can also be found in "Object Catalog". Several (different variants) of this object are among "Basic" morphs.
 
  Explore the assiciated morph-menu. It enables you
  - to toggle showing of "handles". They make it possible to
         - reposition already existing vertices (by moving yellow handles)
         - create new vertices (by moving green handles)
         - delete already existing vertices (by dragging and dropping one yellow handle closely
           nearby the adjacent yellow handle
    Handles can be made visible/hidden by shift+leftclicking the morph. This way it is possible
    to quickly show handles, adjust vertices and then again hide handles.
  - making closed polygon open, i.e. converting it to a curve (and vice versa)
  - toggle smoothed/segmented line/outline
  - set up custom dashing (for line, curves or borders of closed polygons
  - set up custom arrow-heads (for lines resp. curves)
 
  ------------------------------------------------------------------------------------------
  Implementation notes:
 
  This class combines the old Polygon and Curve classes.
 
  The 1-bit fillForm to make display and containment tests reasonably fast.  However, this functionality is in the process of being supplanted by balloon capabilities, which should eventually provide anti-aliasing as well.
 
  wiz 7/18/2004 21:26
  s have made some changes to this class to
 
  1) correct some bugs associated with one vertex polygons.
 
  2) prepare for some enhancements with new curves.
 
  3) add shaping items to menu.!

Item was changed:
  ----- Method: PolygonMorph>>addCustomMenuItems:hand: (in category 'menu') -----
  addCustomMenuItems: aMenu hand: aHandMorph
         "Add morph-specific items to the given menu which was invoked by the given hand.  This method provides is invoked both from the halo-menu and from the control-menu regimes."
 
         super addCustomMenuItems: aMenu hand: aHandMorph.
         aMenu addUpdating: #handlesShowingPhrase target: self action: #showOrHideHandles.
         vertices size > 2 ifTrue:
                 [aMenu addUpdating: #openOrClosePhrase target: self action: #makeOpenOrClosed].
 
         aMenu addUpdating: #smoothPhrase target: self action: #toggleSmoothing.
+        aMenu addUpdating: #snapToOtherPolygonPhrase target: self action: #toggleSnapToOtherPolygons.
         aMenu addLine.
         aMenu add: 'specify dashed line' translated action:  #specifyDashedLine.
 
         self isOpen ifTrue:
                 [aMenu addLine.
                 aMenu addWithLabel: '---' enablement: [self isOpen and: [arrows ~~ #none]] action:  #makeNoArrows.
                 aMenu addWithLabel: '-->' enablement: [self isOpen and: [arrows ~~ #forward]] action:  #makeForwardArrow.
                 aMenu addWithLabel: '<--' enablement: [self isOpen and: [arrows ~~ #back]] action:  #makeBackArrow.
                 aMenu addWithLabel: '<->' enablement: [self isOpen and: [arrows ~~ #both]] action:  #makeBothArrows.
                 aMenu add: 'customize arrows' translated action: #customizeArrows:.
                 (self hasProperty: #arrowSpec)
                         ifTrue: [aMenu add: 'standard arrows' translated action: #standardArrows]].!

Item was changed:
  ----- Method: PolygonMorph>>dropVertex:event:fromHandle: (in category 'editing') -----
+ dropVertex: ix event: evt fromHandle: handle
+        "Leave vertex in new position. If dropped ontop another vertex delete
+        this one.
- dropVertex: ix event: evt fromHandle: handle
-        "Leave vertex in new position. If dropped ontop another vertex delete this one.
         Check for too few vertices before deleting. The alternative
+        is not pretty -wiz"
+        | p world |
-                                is not pretty -wiz"
-        | p |
         p := vertices at: ix.
+       
+        "snap the dropped vertex to a vertex of another PolygonMorph if it is in near proximity"
+        self isSnappingToOtherPolygons
+                ifTrue: [world := Project current world.
+                        world submorphs
+                                do: [:each | ((each respondsTo: #vertices)
+                                                        and: [each ~= self])
+                                                ifTrue: [each vertices
+                                                                do: [:otherMorphsVertex | (otherMorphsVertex dist: p)
+                                                                                        < 3
+                                                                                ifTrue: [vertices at: ix put: otherMorphsVertex]]]]].
         (vertices size >= 2
                         and: ["check for too few vertices before deleting. The alternative
                                 is not pretty -wiz"
                                 ((vertices atWrap: ix - 1)
                                                 dist: p)
                                                 < 3
                                         or: [((vertices atWrap: ix + 1)
                                                         dist: p)
                                                         < 3]])
                 ifTrue: ["Drag a vertex onto its neighbor means delete"
+                        self deleteVertexAt: ix].
-                                self deleteVertexAt: ix .].
         evt shiftPressed
                 ifTrue: [self removeHandles]
                 ifFalse: [self addHandles
                         "remove then add to recreate"]!

Item was changed:
  ----- Method: PolygonMorph>>initialize (in category 'initialization') -----
  initialize
  "initialize the state of the receiver"
         super initialize.
  ""
         vertices := Array
                                 with: 5 @ 0
                                 with: 20 @ 10
                                 with: 0 @ 20.
         closed := true.
         smoothCurve := false.
         arrows := #none.
+        snapToOtherPolygons := false.
         self computeBounds!

Item was added:
+ ----- Method: PolygonMorph>>isSnappingToOtherPolygons (in category 'access') -----
+ isSnappingToOtherPolygons
+   ^snapToOtherPolygons ifNil:[ snapToOtherPolygons := false].
+  !

Item was added:
+ ----- Method: PolygonMorph>>toggleSnapToOtherPolygons (in category 'menu') -----
+ toggleSnapToOtherPolygons
+      ^snapToOtherPolygons := snapToOtherPolygons not!







Carpe Squeak!