-
-
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
Add right click menu for main table header #6601
Comments
Hi, we are a group of 5 students from Adelaide University. Our software process improvement class asked us to contribute to an open-source project. We'd love to help with this issue. Is anyone currently working on this issue? If not can we take it? We are all new contributors to open-source projects. If possible could we get some guidance on which part of the code we should look into first. Thanks a lot |
@sj30001 Thanks for your interest in JabRef. Currently no one is working on this issue. more info on context menu: |
Hello, we are a group of 8 students from Harbin Institute of Technology from China. Our Open Source Software Development class asked us to contribute to an open-source project. We chose this issue among the projects provided by Mr.Treude and we are interested in helping with this issue. Is anyone currently working on this issue? If not can we take it? We are all new contributors to open-source projects. If we make some mistakes, if possible would you please point it out? Thank you very much. |
Hi, thanks for your interest in JabRef. Your teacher already announced your team, so you are free to take it. |
Hello, I am a student who comes from study team 6 at Adelaide University. Due to our course Open Source Software Development requirements, our team is focusing on contributing to an open-source project. After discussion, our team agreed that this project is a really excellent open-source project. At the same time, we are very interested in # 6601 this issue and are willing to make positive contributions. We expect to finish this issue and submit our PR before June. Can we work on this issue? If we can, we would appreciate that if you can give us some advice about how to contribute to this issue. Looking forward to your feedback sincerely. |
Yes you are welcome to dive into this issue. Please prepare your workspace according to our dev docs and please read the discussion at #7729 . Have fun coding. Please make a draft pr as soon as possible, even when you are not finished, so we can follow your changes and help early, if there is something not understood. Feel free to use the Gitter chat to ask questions, if you got any. |
Is this issue still being worked on? I've noticed a few other students have looked at it and unsure if it's still taken. |
Seems like it's still open and I don't see anybody else working for it. Give it a shot. 👍 |
Been making progress on this issues. I have been able to access the currently used columns, but my current problem is being able to create objects of the most commonly used columns. Just wondering if there is a way to access/create these MainColumn objects so if they're not currently in the MainTabke I can still access them. |
Just modify or overload the constructor of the MainTableColumnFactory to make it a bit more generic, for instance instead of accessing directly the columnPreferences just provide a column type to recive a perfectly useable MainTableColumn. If you use the column objects in the Right Click Menu you can probably just add them by demand to the column list of the MainTable. But I would not advise you to use the MainTableColumn objects in the Right Click Menu, but just column types (MainTableColumnModel.Type) to avoid useless overhead. |
It would be nice if right-clicking on the main table header (where the column names are), provided a menu which either allowed to edit the viewed columns/fields directly or provided a link to the preferences.
Maybe a list of the currently and most commonly used fields would show up to be activated/deactivated in the right click menu and for more detailed adjustments a reference to the settings would be displayed.
The text was updated successfully, but these errors were encountered: