Numerous registered events

Bug #2020712 reported by hilaire
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Dr. Geo
Fix Released
Medium
Unassigned

Bug Description

Do we needs to register these events below in DrGMorph?

Because a lot are registered, and not clean up. It clutters the (self actionsMaps) in the ActiveModel and slow down the environment.

DrGMoprh>>registerEvents
 mathItem
  ifNotNil: [
   mathItem
    when: #deletedMathItem
    send: #deleteMorph:
    to: self app
    with: self.
   mathItem
    when: #updateMyView
    send: #updateAllMathItems
    to: self domain]

Tags: core
Revision history for this message
hilaire (hilaire-fernandes) wrote :

Implemented something like that in Sketch editor and browser to invoke release of event

DrGSmalltalkSketchEditor>>releaseMiniView
 drgeoView submorphCount = 1 ifTrue: [
  drgeoView firstSubmorph ifNotNil: [:miniView| miniView app release] ]

Changed in drgeo:
status: Confirmed → In Progress
Revision history for this message
hilaire (hilaire-fernandes) wrote :

Observe in the coming release how the Smalltalk Sketch tools behave.

Changed in drgeo:
milestone: wip → 23.06
Revision history for this message
hilaire (hilaire-fernandes) wrote :

But all this event attached to math item and view should be discarded

Revision history for this message
hilaire (hilaire-fernandes) wrote :

Set/Search for code with flag: #drgeoEvent

Changed in drgeo:
milestone: 23.06 → 23.12
Changed in drgeo:
status: In Progress → Fix Committed
Changed in drgeo:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.