-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Non-editable user comments #10870
Comments
can you please test the latest development version? I can no longer reproduce the issue |
I still can reproduce it, yes. |
Hi @Siedlerchr, I would like to work on this issue. |
Sure, you're welcome to take a look at this issue |
As a general advice for newcomers: check out Contributing for a start. Also, guidelines for setting up a local workspace is worth having a look at. Feel free to ask here at GitHub, if you have any issue related questions. If you have questions about how to setup your workspace use JabRef's Gitter chat. Try to open a (draft) pull-request early on, so that people can see you are working on the issue and so that they can see the direction the pull request is heading towards. This way, you will likely receive valuable feedback. |
Hi @ror3d, can you tell me what exactly your username is? Is it |
Hi, Is that the setting where the name is taken from? it doesn't feel like it is, since the field is disabled unless the "Mark new entries" checkbox is selected. If that is indeed the setting, maybe it should be always editable and made more clear that it's what is used also for the personal comments. Maybe @Siedlerchr can weigh in on that. Anyway, in that field I have it as |
Thanks for clarifying, @ror3d. The issue will be fixed in the next release. There was an issue if the owner name had uppercase or special characters. Yes, this setting is to override the default owner name (which comes from your OS username), check the "Mark new entries" checkbox. This will allow you to edit the field and change the owner name. |
I agree with you. Originally, it's intended to be used for new entries to add the owner field. Now it's also used for comment fields. This allows sharing your bib files with others and everyone has it's own comment field. It should always be editable, independent of the checkbox |
Sure, I'll add this to the pull request |
Is the original issue really fixed? 😅 |
Indeed, the issue seems to be fixed with the latest code from main. It also fixes the problem where saving after editing the comment was detected as a different program modifying the file, which I guess was a side-effect of this issue. |
JabRef version
5.12 (latest release)
Operating system
Windows
Details on version and operating system
Windows 11
Checked with the latest development build (copy version output from About dialog)
Steps to reproduce the behaviour
I am unsure of the cause for this, as a peer doesn't seem to have this problem.
Comment-<username>
Comment-<username>
field:Appendix
@Article{ArticleID,
comment-roc = {Just a test comment},
comment-roc = {Just a test comment},
}
The text was updated successfully, but these errors were encountered: