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.

Sign up for GitHub

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

Jump to bottom

fix: editor context menu and popup #484

Merged
merged 2 commits into from
Dec 30, 2024
Merged

fix: editor context menu and popup #484

merged 2 commits into from
Dec 30, 2024

Conversation

shczhen
Copy link
Collaborator

@shczhen shczhen commented Dec 30, 2024 •
edited
Loading

#409
#410

Checklist

  • update editor context menu actions
  • update confirmation dialog style
  • add toast to display post-save status

Screenshots

image image image

@shczhen shczhen requested a review from halajohn as a code owner December 30, 2024 10:45
This was linked to issues Dec 30, 2024
In the editor popup, right-click to bring up the context menu, which only needs to include the "save" item for now #409
Closed
The style of the confirmation dialog that appears from the editor popup's context menu needs to be improved to look more polished and professional #410
Closed
@halajohn halajohn merged commit cd72810 into main Dec 30, 2024
15 of 16 checks passed
@halajohn halajohn deleted the fix-fe-409-editor-context-menu branch December 30, 2024 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@halajohn halajohn halajohn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Kanban
Status: Done
Milestone
No milestone
2 participants
@shczhen @halajohn