Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't quickly open classes and tests in the Demo #485

Open
Eliott-Guevel opened this issue Mar 27, 2024 · 1 comment
Open

Can't quickly open classes and tests in the Demo #485

Eliott-Guevel opened this issue Mar 27, 2024 · 1 comment

Comments

@Eliott-Guevel
Copy link

Eliott-Guevel commented Mar 27, 2024

The window opened by the Open Bloc Demo option located in the Help tab shows tests and classes correctly, but there's no option to double-click or right-click and select an option on a class/test in the left column of the window in order to quickly open it in the System Browser.

As reference I implemented this exact feature for a tree (classes and methods) in another project, see here
see the interfaceTreeAdd: interfaces forFrame: interfaceFrame roleInterface: roleInterface method, precisely the whenActivatedDo: part

interfaceTree := self newTree
			                 children: [ :i | interface methodDict keys ];
			                 displayIcon: [ :aClass |
				                 self iconNamed: aClass systemIconName ];
			                 roots: { interface };
			                 expandRoots;
			                 activateOnDoubleClick;
			                 whenActivatedDo: [ :eventServiceParameter |
				                 eventServiceParameter isTrait
					                 ifTrue: [
						                 eventServiceParameter selectedItem browse ]
					                 ifFalse: [
						                 compiledMethod := CompiledMethod new.
						                 compiledMethod methodClass: interface.
						                 compiledMethod selector: eventServiceParameter.
						                 compiledMethod browse ] ].
@Eliott-Guevel
Copy link
Author

Eliott-Guevel commented Mar 27, 2024

And I also noticed that the canva variable of BlDemoPresenter should be spelled canvas
(the initializeCanvaPresenter method sould also be renamed initializeCanvasPresenter)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant