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

fix: Add a empty state msg when there is no recently opened files #4292

Closed
wants to merge 1 commit into from

Conversation

holygeek00
Copy link
Contributor

Closes #4119

Screenshot 2024-05-14 at 14 40 12

@holygeek00 holygeek00 requested review from marstamm and a team May 14, 2024 12:40
@holygeek00 holygeek00 self-assigned this May 14, 2024
@holygeek00 holygeek00 requested review from philippfromme and removed request for a team May 14, 2024 12:40
@bpmn-io-tasks bpmn-io-tasks bot added the needs review Review pending label May 14, 2024
Copy link

This Pull Request targets develop branch, but contains fix commits.

Consider targeting main instead.

@@ -210,7 +210,7 @@ class MenuBuilder {
enabled: true,
submenu: Menu.buildFromTemplate([
{
label: 'Reopen Last File',
label: this.options.state.closedTabs.length > 0 ? 'Reopen Last File' : 'No Files Opened',
Copy link
Member

@nikku nikku May 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's ensure this works with a blank setup (empty APP_DATA/config.json) (cf. docs).

Suggested change
label: this.options.state.closedTabs.length > 0 ? 'Reopen Last File' : 'No Files Opened',
label: this.options.state.closedTabs?.length > 0 ? 'Reopen Last File' : 'No Files Opened',

You can try to clear the Modeler application data and see the impact.

@nikku nikku closed this May 27, 2024
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label May 27, 2024
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.

Add a empty state msg when there is no recently opened files
2 participants