Martín Dias

Results 460 comments of Martín Dias

All the `Br*` seem plausible to be in Brick. I checked the references of each class and the only that looked non-trivial was `BrValuableStencil`which is referenced from `TBlElementError>>#defaultErrorStencil` in `BlocPac`.

This would be fixed if `BlMouseProcessor>>#canBeClickEvent:`, which is defined as: ``` canBeClickEvent: aMouseUpEvent ^ dndInProgress not and: [ (previousMouseDownEvent isNotNil and: [ previousMouseDownEvent primaryButtonPressed ]) and: [ aMouseUpEvent target =...

I see the class comment `BlMouseClick` now: ``` The click event is fired when a pointing device button (a mouse's primary button) is pressed and released on a single element....

Also, it may me convenient (to ease use) to transform BlMouseClick into a hierarchy of ``` BlMouseClick BlPrimaryMouseClick BlMiddleMouseClick BlSecondaryMouseClick ``` the names would be consistent with the hierarchy of...

BTW, the message selector is `requestDropReference:`.

Related: Problems in the method `BlMorphicWorldMorph>>#beMain` - Nonexistent method: `Cursor class >> currentCursor:` - Nonexistent global variable: `ActiveHand`

Another related: The method `BlMorphicWorldState>>#assuredCanvas` sends `self canvas` and `self canvas: ...` and they are not implemented in the hierarchy. It's strange that its sibling `AthensWorldState` has `#doDeferredUpdatingFor:` with same...

Another one, related too: The `BlocHost-Morphic` package extends `PharoCommonTools` with a method that references an nonexistent class `Transcripter`: ```smalltalk emergencyDebugger ^ tools at: #emergencyDebugger ifAbsent: [ Transcripter ] ```

Sub-issues: - https://github.com/pharo-graphics/Bloc/issues/141 - https://github.com/pharo-graphics/Bloc/issues/142

By a full string search in a text editor, I see that BlElementSelection belongs to category 'Bloc-DevTool-Core'.