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

Added error dialog when setting invalid main file directory #1921

Merged
merged 1 commit into from Sep 11, 2016

Conversation

boceckts
Copy link
Contributor

@boceckts boceckts commented Sep 5, 2016

Many users set file directories by copy pasting the path and not by opening the file chooser. This means invalid paths could be set in preferences -> File -> main file directory

See koppor#109 for more information.

Now the preferences tab won't save an invalid path anymore and notifies the user about it.
Also there is now an error message when trying to download a file to a directory which does not exist or can not be written to.

errormainfiledirectory
filedownloadproblem

  • Change in CHANGELOG.md described
  • Tests created for changes
  • Screenshots added (for bigger UI changes)
  • Manually tested changed features in running JabRef
  • Check documentation status (Issue created for outdated help page at help.jabref.org?)

@tschechlovdev tschechlovdev added status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers and removed stupro-ready-for-internal-review labels Sep 9, 2016
@boceckts boceckts changed the title [WIP] Added error dialog when setting invalid main file directory Added error dialog when setting invalid main file directory Sep 9, 2016
@tobiasdiez
Copy link
Member

LGTM 👍

@stefan-kolb stefan-kolb merged commit 35b35dc into JabRef:master Sep 11, 2016
@boceckts boceckts deleted the fixFileNotFoundException branch September 11, 2016 11:18
Siedlerchr added a commit that referenced this pull request Sep 11, 2016
* master:
  Remove obsolete wrapper task
  Added error dialog when setting invalid main file directory (#1921)
  Add filteringCharset = 'UTF-8' (#1945)
  Include https://github.com/grimes2
  Searchbar across all bib files instead each having its own (#1549)
  Some OO/LO cleanups (#1927)
  Update link
  Removed external dependency in logic (#1934)
Siedlerchr added a commit to Siedlerchr/jabref that referenced this pull request Sep 11, 2016
* master:
  Remove obsolete wrapper task
  Added error dialog when setting invalid main file directory (JabRef#1921)
  Add filteringCharset = 'UTF-8' (JabRef#1945)
  Include https://github.com/grimes2
  Searchbar across all bib files instead each having its own (JabRef#1549)
  Some OO/LO cleanups (JabRef#1927)
  Update link
  Removed external dependency in logic (JabRef#1934)
zesaro pushed a commit to zesaro/jabref that referenced this pull request Nov 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants