Skip to content

Delete showSpace: in BlElement and BlSpace? #698

Closed
@tinchodias

Description

@tinchodias

There are 2 implementors of showSpace: but zero senders. I have not only Bloc and Toplo loaded, also Pyramid and other projects loaded in my image.

Implementor in BlSpace:

showSpace: aSpace
	"I show a given space either inside of me or let it open by default"
	aSpace show

In BlElement:

showSpace: aSpace
	self hasParent
		ifTrue: [ self parent showSpace: aSpace ]
		ifFalse: [
			self isAttachedToSceneGraph
				ifTrue: [ self space showSpace: aSpace ]
				ifFalse: [ aSpace show ] ]

It seems the intention is to support a scenario of inner vs independent windows, via hooks (overriding one of such implementors).

Do you have an opinion @plantec ?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions