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

Add Toplo world menu entry #16429

Conversation

tinchodias
Copy link
Contributor

No description provided.

Copy link

request-info bot commented Apr 9, 2024

This issue has either a default title or empty body. We would appreciate it if you could provide more information. Note: I am not a very intelligent bot, I can only react to new comments. Please add a comment for me if you update the body or title.

@tesonep
Copy link
Collaborator

tesonep commented Apr 10, 2024

Fix #16289

@tesonep tesonep merged commit c2ef904 into pharo-project:Pharo12 Apr 10, 2024
1 of 2 checks passed
@tinchodias
Copy link
Contributor Author

I just downloaded P12 via zeroconf and don't see the menu entry. After 6 hours. I have to wait more or it was reverted?

Build information: Pharo-12.0.0+SNAPSHOT.build.1474.sha.723e992378842828924aea037a6d2da975cd8da4 (64 Bit)

@Rinzwind
Copy link
Contributor

It wasn’t reverted, but in the build that was triggered for the commit in which this pull request was merged (build 1475), the ‘Full Image-64’ stage failed due to issue #16443, and while the issue seems to have been fixed, there hasn’t been another build yet.

@Rinzwind
Copy link
Contributor

It would have been nice if the change to Spec that seems to fix issue #16443 would have triggered a new build so that there would be an image with the changes of this pull request, I have opened an issue about that: issue #16452.

@Rinzwind
Copy link
Contributor

@tinchodias: There are images for newer builds now. There’s no Toplo world menu item as far as I can tell though. The class BlLoadProjectWorldMenu seems to be missing methods with a <worldMenu> pragma.

@tinchodias
Copy link
Contributor Author

@Rinzwind many thanks! Silly me, I forgot to include a method in the PR changes. Luckily I located the image where I worked that day and was able to push the missing method addition (PR #16478)

@Rinzwind
Copy link
Contributor

Ah OK! I wasn’t quite sure whether the <worldMenu> pragma method was missing or whether the menu item was supposed to get added in some other way that I was not seeing.

I wanted to take a look because I was wondering whether the menu item would load a version including the world renderer canvas scale factor support and the Toplo demo browser which I added in Bloc pull request #465 and Toplo pull request #120.

@tinchodias
Copy link
Contributor Author

@Rinzwind This menu entry loads the Pharo12 branch of Toplo, and we will take care it loads a reasonably good version of Toplo and Bloc, and should have a stable version that works well in Pharo 12 release and can update from time to time to a more recent version.

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

Successfully merging this pull request may close these issues.

None yet

3 participants