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

Add a method to create a new link manually #11017

Closed
ror3d opened this issue Mar 13, 2024 · 8 comments · Fixed by #11539
Closed

Add a method to create a new link manually #11017

ror3d opened this issue Mar 13, 2024 · 8 comments · Fixed by #11539
Assignees
Labels

Comments

@ror3d
Copy link
Contributor

ror3d commented Mar 13, 2024

Is your suggestion for improvement related to a problem? Please describe.
Currenty, the General tab > File entries can hold either paths to local files, or links to online URLs, but when creating them manually, only local files can be added there (either by selecting a local file with the file chooser, or by downloading a URL).

This makes it difficult to use that field for other information such as e.g. links to github repositories for code related to the article.

Describe the solution you'd like
It would be good if a method to create a new file entry manually was available -- open the file entry editor with blank entries, allow filling them up manually, and create the entry with the result of that operation.

@rahulsree
Copy link

Hi, I'm interested in contributing to this issue, I'd appreciate if you could provide me with the assignee role.

@koppor koppor added the FirstTimeCodeContribution Triggers GitHub Greeter Workflow label Mar 14, 2024
@ThiloteE ThiloteE moved this from Free to take to Reserved in Candidates for University Projects Mar 14, 2024
@ThiloteE ThiloteE moved this from Free to take to Reserved in Good First Issues Mar 14, 2024
@koppor
Copy link
Member

koppor commented Apr 8, 2024

@rahulsree May I ask for progress?

@ThiloteE ThiloteE moved this from Reserved to Free to take in Candidates for University Projects May 20, 2024
@ThiloteE ThiloteE moved this from Reserved to Free to take in Good First Issues May 20, 2024
@koppor koppor removed the FirstTimeCodeContribution Triggers GitHub Greeter Workflow label Jul 3, 2024
@JabRef JabRef deleted a comment from github-actions bot Jul 3, 2024
@Kunal77689
Copy link
Contributor

Kunal77689 commented Jul 24, 2024

Hi, I'd like to work on this issue, Could you please assign it to me?

@ThiloteE ThiloteE moved this from Free to take to Reserved in Candidates for University Projects Jul 24, 2024
@ThiloteE ThiloteE moved this from Free to take to Reserved in Good First Issues Jul 24, 2024
@ThiloteE ThiloteE added the FirstTimeCodeContribution Triggers GitHub Greeter Workflow label Jul 24, 2024
Copy link
Contributor

Welcome to the vibrant world of open-source development with JabRef!

Newcomers, we're excited to have you on board. Start by exploring our Contributing guidelines, and don't forget to check out our workspace setup guidelines to get started smoothly.

Having any questions or issues? Feel free to ask here on GitHub. Need help setting up your local workspace? Join the conversation on JabRef's Gitter chat. And don't hesitate to open a (draft) pull request early on to show the direction it is heading towards. This way, you will receive valuable feedback.

Happy coding! 🚀

Kunal77689 added a commit to Kunal77689/jabref that referenced this issue Jul 25, 2024
Manual File Entry Creation:
  Added a new button to the LinkedFilesEditor.fxml.
  Clicking this button displays a textbox where users can manually enter text/URLs. [JabRef#11017]
@koppor
Copy link
Member

koppor commented Jul 25, 2024

As screenshot:

2024-07-26_00-19
  • Instead of directly opening the file browse dialog, the linked file dialog should be opened
  • The dialog has a different heading and a different finish button (see screenshot)
  • The old logic directly opening a file browser is to be removed -> this dialog is the only thing shown.

We accept that for attaching files, now three clicks are necessary. We gain the benefit that the user can directly add a description.

@Kunal77689
Copy link
Contributor

Kunal77689 commented Jul 26, 2024

@koppor What file types can we expect to be available in the combo box?

@koppor
Copy link
Member

koppor commented Jul 26, 2024

@koppor What file types can we expect to be available in the combo box?

The whole dialogue including the combobox is already available. One "just" needs to adapt it to cover two cases: add (new), edit (existing)

@koppor
Copy link
Member

koppor commented Jul 26, 2024

@Kunal77689 please open jabref, create a new library, drag and drop a PDF into it. Hold the ctrl key, otherwise it will be moved. Then go to "General" tab, Files. There you will see the pdf. Then click the edit button on the right. You will see the dialog.

Kunal77689 added a commit to Kunal77689/jabref that referenced this issue Jul 27, 2024
Manual File Entry Creation:
  Modified add button functionality so that rather than opening the file explorer, it opens a dialog box for manual file creation. [JabRef#11017]
Kunal77689 added a commit to Kunal77689/jabref that referenced this issue Jul 27, 2024
- Deleted fxml and java files which were handling edit and add action separately and added a unified way which handles edit and add action dynamically
Kunal77689 added a commit to Kunal77689/jabref that referenced this issue Jul 28, 2024
- Overloaded the constructor so that one will take a LinkedFile for editing, another will not take any argument.
Kunal77689 added a commit to Kunal77689/jabref that referenced this issue Jul 31, 2024
- Removed extra blank line in LinkedFilesEditorViewModel
-Added link to the pr in the CHANGELOG.md
Kunal77689 added a commit to Kunal77689/jabref that referenced this issue Jul 31, 2024
Kunal77689 added a commit to Kunal77689/jabref that referenced this issue Jul 31, 2024
Kunal77689 added a commit to Kunal77689/jabref that referenced this issue Jul 31, 2024
- Fixed findObsoleteLocalizationKeys() error by removing unwanted keys.
- Fixed findMissingLocalizationKeys() by adding "Add file link" as a key
Kunal77689 added a commit to Kunal77689/jabref that referenced this issue Aug 6, 2024
- Renamed files from controller->view
- Made sure both edit and add dialog boxes follow   same layout
- The add dialog box will have pdf as the default   option
- fixed minor bugs in some places where some "%"    were removed in some of the previous commits,     restored those symbols
- on hover, rather than showing open, it will show add
github-merge-queue bot pushed a commit that referenced this issue Aug 6, 2024
* Add a method to create a new link manually #11017
Manual File Entry Creation:
  Modified add button functionality so that rather than opening the file explorer, it opens a dialog box for manual file creation. [#11017]

* Updated CHANGELOG.md

* Added a method to create a new link manually #11017

- Deleted fxml and java files which were handling edit and add action separately and added a unified way which handles edit and add action dynamically

* Added a method to create a new link manually #11017

- Overloaded the constructor so that one will take a LinkedFile for editing, another will not take any argument.

* Added a method to create a new link manually #11017

- Removed extra blank line in LinkedFilesEditorViewModel
-Added link to the pr in the CHANGELOG.md

* Added a method to create a new link manually #11017

- Fixed checkstyle erorrs

* Added a method to create a new link manually #11017

- Fixed checkstyle erorrs

* Added a method to create a new link manually #11017

- Fixed findObsoleteLocalizationKeys() error by removing unwanted keys.
- Fixed findMissingLocalizationKeys() by adding "Add file link" as a key

* Added a method to create a new link manually #11017

- Renamed files from controller->view
- Made sure both edit and add dialog boxes follow   same layout
- The add dialog box will have pdf as the default   option
- fixed minor bugs in some places where some "%"    were removed in some of the previous commits,     restored those symbols
- on hover, rather than showing open, it will show add

* Use Java 8 Optional

* Fix class names

* Fix name (again)

* Fix method name and reuse code

* Fix double dialog initialization

* Fix PDF type

* Remove generated JavaDoc

* Remove dead code

* Re-add missing localizations

---------

Co-authored-by: Oliver Kopp <kopp.dev@gmail.com>
@github-project-automation github-project-automation bot moved this from Reserved to Done in Good First Issues Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants