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

Delete from disk should not be default option/selected on DELETE in file tab #3679

stefan-kolb opened this issue Jan 31, 2018 · 1 comment


Copy link

@stefan-kolb stefan-kolb commented Jan 31, 2018


Changing the buttons should be sufficient.

Copy link

@bdach bdach commented Feb 3, 2018

Hi! 👋

I have submitted a pull request (#3690) aiming to fix this issue (and another issue with the "Cancel" button, which I discovered during testing my first change). I would appreciate it if you could review the changes and suggest possible improvements!

tobiasdiez added a commit that referenced this issue Feb 5, 2018
* Only remove file links from entries by default (#3679)

When a file link in a bibliography entry is deleted by the user,
a dialog is displayed. Previously, the default (first) dialog option
deleted the linked file from disk. This commit changes the default
behaviour to just removing the file link from the entry.

* Fix canceling bug in file link deletion dialog

When clicking "Cancel" button in the file link deletion dialog, the file
link was being deleted from the list. This commit fixes this behaviour
by changing the return value of the delete() method when the "Cancel"
button is chosen.

* Tag link removal button as default (#3679)

The "remove from entry" option in the file link removal dialog window
has been tagged with the ButtonData.YES type to ensure correct
positioning and behaviour on all platforms.

* Add test cases for linked file removal dialog

* Code style adjustments in dialog test cases

* Added non-empty link to file in one test for clarity
* Removed unnecessary newlines
* Changed test method names to adhere to chosen naming convention
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants