From d5d19c439140a629248a1a53ad82ece1623e6e0e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jan=20Blizni=C4=8Denko?= Date: Mon, 22 Jul 2024 18:46:13 +0200 Subject: [PATCH] Added few comments, removed unused method --- repository/OpenPonk-Core/OPController.class.st | 6 +++++- repository/OpenPonk-Core/OPDiagramController.class.st | 8 +------- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/repository/OpenPonk-Core/OPController.class.st b/repository/OpenPonk-Core/OPController.class.st index 502f2855..71aaba5e 100644 --- a/repository/OpenPonk-Core/OPController.class.st +++ b/repository/OpenPonk-Core/OPController.class.st @@ -71,6 +71,8 @@ OPController >> dependentControllers [ { #category : 'accessing' } OPController >> dependentElements [ + "model elements that require this one to be rendered in order to render itself" + ^ self elementsToShowInside ] @@ -124,7 +126,9 @@ OPController >> diagramElement: aDiagramElement [ { #category : 'accessing' } OPController >> elementsToShowInside [ - ^ #() + "model elements that should be shown (rendered) automatically when this one is rendered" + + ^ #( ) ] { #category : 'accessing' } diff --git a/repository/OpenPonk-Core/OPDiagramController.class.st b/repository/OpenPonk-Core/OPDiagramController.class.st index 26fed074..770f401c 100644 --- a/repository/OpenPonk-Core/OPDiagramController.class.st +++ b/repository/OpenPonk-Core/OPDiagramController.class.st @@ -84,8 +84,7 @@ OPDiagramController >> addMouseEventsFor: aFigure [ { #category : 'rendering' } OPDiagramController >> allShowableElements [ - - "Should return all elements that could be directly hidden on diagram" + "all elements that could be directly hidden on diagram" ^ (Set with: model) , self elementsToShowInside ] @@ -548,11 +547,6 @@ OPDiagramController >> modelForFigure: aFigure [ ^ (self controllerForRoassalShape: aFigure) model ] -{ #category : 'events handling' } -OPDiagramController >> modelRemoved: aModel [ - self controllerForModel: aModel ifFound: #removeController -] - { #category : 'accessing' } OPDiagramController >> modelToControllerMapping [ self