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

No confirmation dialog when overwriting files with exported images #4764

Open
falko opened this issue Dec 16, 2024 · 3 comments
Open

No confirmation dialog when overwriting files with exported images #4764

falko opened this issue Dec 16, 2024 · 3 comments
Labels
backlog Queued in backlog bug Something isn't working good first issue Good for newcomers platform:Linux spring cleaning Could be cleaned up one day ux

Comments

@falko
Copy link
Member

falko commented Dec 16, 2024

Describe the bug

There seems to be no confirmation dialog when overwriting files with exported images and that made me question if the repeated export worked and I selected the right folder.

Steps to reproduce

  1. Export an image from a BPMN model
  2. Export an image from a BPMN model into the same folder with the same name, e.g. keeping the suggested file name during both exports
  3. The file gets overwritten without confirmation

Expected behavior

Confirmation requested before overwriting files

Environment

Camunda Modeler system information

  • Version: 5.31.0-nightly.20241201
  • Operating System: Linux x86_64 amd64
  • Plugins: c8-data-outline-plugin, bpmn-js Token Simulation
  • Execution Platform: BPMN - Camunda 8

Additional context

I was exporting different versions of a customer model that happened to have the same file name. In my case, I wanted to overwrite the file but I imagine in most other cases users would at least like to be asked.

@falko falko added the bug Something isn't working label Dec 16, 2024
@barmac
Copy link
Collaborator

barmac commented Dec 17, 2024

I cannot reproduce it on MacOS. @nikku can you check on Linux?
image

@barmac barmac added the needs more information Requires additional information to be actionable. label Dec 18, 2024
@nikku
Copy link
Member

nikku commented Dec 19, 2024

I can reproduce the issue on Linux (Ubuntu). This override notification works in other editors that I use, so this is a valid issue, and not "platform specific behavior".

Attaching spring cleaning and ux labels.

Thanks for reporting @falko.

@nikku nikku added ux spring cleaning Could be cleaned up one day platform:Linux backlog Queued in backlog good first issue Good for newcomers and removed needs more information Requires additional information to be actionable. labels Dec 19, 2024
@falko
Copy link
Member Author

falko commented Dec 19, 2024

Thx for looking into it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog bug Something isn't working good first issue Good for newcomers platform:Linux spring cleaning Could be cleaned up one day ux
Projects
None yet
Development

No branches or pull requests

3 participants