Skip to content

Show the gameplay test run button at the center of the toolbar - #9041

Merged
4ian merged 1 commit into
masterfrom
gameplat-test-toolbar
Sep 1, 2026
Merged

Show the gameplay test run button at the center of the toolbar#9041
4ian merged 1 commit into
masterfrom
gameplat-test-toolbar

Conversation

@Bouh

@Bouh Bouh commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Description

montage-mtipq0eg.mp4

The button used to run a gameplay test was displayed at the right of the toolbar, next to the properties panel icon, while every other editor shows its preview button at the center. It is now displayed in its own centered toolbar group, taking the place of the preview button.

The properties panel had a second "Run the test" button, which is removed: a single button remains, always visible even when the panel is closed. The panel keeps the progress indicator shown while a test is running.

Changes

  • GameplayTestEditorToolbar: the run/stop button moves to its own centered ToolbarGroup, the properties panel icon stays in the trailing group.
  • MainFrame/Toolbar: the empty placeholder ToolbarGroup is not rendered when there is no preview button, so the editor toolbar can provide the centered group itself. This only concerns the gameplay test editor, the only one setting showPreviewAndShareButtons to false while showing the project buttons.
  • GameplayTestProperties: the duplicated run/stop buttons are removed, along with the now unused onRunTest and onStopTest props (and their plumbing in GameplayTestEditor, GameplayTestEditorContainer and the story).

Checklist

  • Prettier and ESLint pass on the changed files.

The button now takes the place of the preview button of the other
editors, and is the only way to run or stop a test: the duplicated
button of the properties panel is removed, so that a test can still
be run when the panel is closed.
@Bouh
Bouh requested a review from 4ian as a code owner September 1, 2026 13:21
@4ian
4ian merged commit 75dbe8e into master Sep 1, 2026
7 checks passed
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

Successfully merging this pull request may close these issues.

2 participants