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

(O3-2482):show save confirmation modal to rename attachments #7

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

jona42-ui
Copy link
Collaborator

@jona42-ui jona42-ui commented Oct 2, 2023

Objective

Allowing Users to Provide Meaningful Names for Saved Attachments

Description:

This enhancement aims to provide users with the ability to specify a meaningful or custom name when saving attachments in the SvgEditor. Currently, attachments are saved with auto-generated filenames, which may not be user-friendly. This enhancement will improve the user experience by allowing users to name their attachments based on their context.

Changes Made:

  • Added a modal dialog that prompts users to enter an attachment name before saving.
  • The attachment name provided by the user will be used as the filename when saving the annotated image as an attachment.
  • Updated the save functionality to use the user-provided attachment name as the filename.

Screenshots :

Screenshot from 2023-10-02 14-12-50

Related Issues:

O3-2482

Additional Notes:

@jona42-ui
Copy link
Collaborator Author

Screencast.from.10-05-2023.01.12.10.PM.webm

@ibacher @denniskigen is this what you would expect

cc: @gracepotma

@jona42-ui jona42-ui changed the title O3-2482; Allowing Users to Provide Meaningful Names while saing annotated Attachments (O3-2482:show save confirmation modal to rename attachments Oct 30, 2023
@jona42-ui jona42-ui changed the title (O3-2482:show save confirmation modal to rename attachments (O3-2482):show save confirmation modal to rename attachments Oct 30, 2023
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.

1 participant