| Anonymous | Login | 08-06-2020 22:51 UTC |
| Main | My View | View Issues | Change Log | Docs |
| Viewing Issue Advanced Details [ Jump to Notes ] | [ View Simple ] [ Issue History ] [ Print ] | ||||||||
| ID | Category | Severity | Reproducibility | Date Submitted | Last Update | ||||
| 0001454 | [Squeak] Morphic | tweak | always | 07-13-05 07:24 | 02-15-06 19:01 | ||||
| Reporter | wiz | View Status | public | ||||||
| Assigned To | |||||||||
| Priority | normal | Resolution | fixed | Platform | |||||
| Status | closed | OS | |||||||
| Projection | none | OS Version | |||||||
| ETA | none | Fixed in Version | 3.9 | Product Version | |||||
| Product Build | |||||||||
| Summary | 0001454: [Fix] Add vertex handle doesn't work for arrows. | ||||||||
| Description |
Go to objects. Get an arrow. Add handles. The triangle does not add a vertex. Huh? |
||||||||
| Steps To Reproduce | |||||||||
| Additional Information |
Polygon class ArrowPrototype goes thru pains to make the arrow inflexible. This confuses the users who expect the add vertex triangle to work as normal. This changeset comments out the old code and reverts to the more usual behavior. Ideally someone should do a use test to see which way is best for users. Realistically someone should choose whether this is a bug and fix it or not. Historical notes: the routines affected were changed in sept-oct 2002 by sw. This changeset is offered to give the maintainers a choice and to give the users who don't like the current behavior a remedy."! !PolygonMorph methodsFor: 'editing' stamp: 'wiz 7/13/2005 00:11'! newVertex: ix event: evt fromHandle: handle "Insert a new vertex and fix everything up!! Install the drag-handle of the new vertex as recipient of further mouse events." | pt | "(self hasProperty: #noNewVertices) ifFalse: [pt _ evt cursorPoint. self setVertices: (vertices copyReplaceFrom: ix + 1 to: ix with: (Array with: pt)). evt hand newMouseFocus: (handles at: ((ix + 1) * 2) - 1)]" "modified to remove now vestigial test. see PolygonMorph class>>arrowprototype" pt _ evt cursorPoint. self setVertices: (vertices copyReplaceFrom: ix + 1 to: ix with: (Array with: pt)). evt hand newMouseFocus: (handles at: ((ix + 1) * 2) - 1)! ! |
||||||||
| Attached Files |
 arrowPrototypeFix-wiz.1.cs.gz [^] (793 bytes) 07-13-05 07:24 arrowPrototypeFix-wiz.2.cs.gz [^] (1,063 bytes) 07-13-05 07:44 |
||||||||
|
|
|||||||||
Notes |
|
|
(0001762 - 636 - 1106 - 1106 - 1106 - 1106 - 1106) wiz 07-13-05 07:47 |
First cs missed the change to PM class>>arrowPrototype. While either change fixes the behavior, keeping the code clean requires both fixes. !PolygonMorph class methodsFor: 'instance creation' stamp: 'wiz 7/13/2005 00:43'! arrowPrototype "Answer an instance of the receiver that will serve as a prototypical arrow" | aa | aa _ self new. aa vertices: (Array with: 0@0 with: 40@40) color: Color black borderWidth: 2 borderColor: Color black. "aa setProperty: #noNewVertices toValue: true." "Revert to expected behavior. Remove vestigial code." aa makeForwardArrow. "is already open" aa computeBounds. |
|
(0003870 - 697 - 799 - 799 - 799 - 799 - 799) MarcusDenker 02-15-06 19:01 |
in 3.9 Name: MorphicExtras-jmv.9 Author: jmv Time: 12 January 2006, 1:33:41 pm UUID: 2720167c-765e-9541-8f14-dd3789d8fca8 Ancestors: MorphicExtras-CdG.7 This version includes stuff from one or more of: Mantis-1092-MorphDropFix, fix by Edgar De Cleene (edc) and Scott Wallace (sw) Mantis-0503-TargetSighting, fix by Jerome Peace (wiz) Mantis-1015-SnapView, fix by Jerome Peace (wiz) Mantis-1771-ClickExerciser, fix by Jerome Peace (wiz) Mantis-1625-Thumbnails, fix by Jerome Peace (wiz) Mantis-1484-TrashHalo, fix by Jerome Peace (wiz) Mantis-1454-ArrowPrototypeFix, fix by Jerome Peace (wiz) Mantis-1347-ListDoubleClick, fix by Jerome Peace (wiz) Reviewed by Juan Vuletich (jmv) |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 07-13-05 07:24 | wiz | New Issue | |
| 07-13-05 07:24 | wiz | File Added: arrowPrototypeFix-wiz.1.cs.gz | |
| 07-13-05 07:44 | wiz | File Added: arrowPrototypeFix-wiz.2.cs.gz | |
| 07-13-05 07:47 | wiz | Note Added: 0001762 | |
| 07-13-05 19:09 | KenCausey | Relationship added | duplicate of 0001448 |
| 02-15-06 19:01 | MarcusDenker | Status | new => closed |
| 02-15-06 19:01 | MarcusDenker | Note Added: 0003870 | |
| 02-15-06 19:01 | MarcusDenker | Resolution | open => fixed |
| 02-15-06 19:01 | MarcusDenker | Fixed in Version | => 3.9 |
| Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
49 total queries executed. 35 unique queries executed. |  |
