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

Scene deleting #114

Merged
merged 10 commits into from
Oct 21, 2022
Merged

Scene deleting #114

merged 10 commits into from
Oct 21, 2022

Conversation

Dgzt
Copy link
Collaborator

@Dgzt Dgzt commented Oct 19, 2022

Added submenus to scene menu.

  • Open - Opens the selected scene
  • Delete - Deletes the scene

image

The Delete button is disabled on the current selected scene:

image

After clicked to the Delete button then the application shows a confirmation modal:

image

@Dgzt Dgzt linked an issue Oct 19, 2022 that may be closed by this pull request
@Dgzt Dgzt marked this pull request as ready for review October 20, 2022 04:51
@Dgzt Dgzt requested a review from JamesTKhan October 20, 2022 04:51
@Dgzt
Copy link
Collaborator Author

Dgzt commented Oct 20, 2022

I will add Rename button between Open and Delete in the next merge request.

Copy link
Owner

@JamesTKhan JamesTKhan left a comment

Choose a reason for hiding this comment

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

This looks good, and very useful feature.

@JamesTKhan JamesTKhan added this to the 0.5.0 Release milestone Oct 21, 2022
@JamesTKhan JamesTKhan merged commit 3ff31e2 into JamesTKhan:master Oct 21, 2022
@Dgzt Dgzt deleted the scene-deleting branch October 24, 2022 06:05
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.

Scene deleting
2 participants