-
-
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
Make Show Preferences Window searchable #1019
Comments
It's a nice idea however I'd rather have it in the preferences window itself than in the I'd close this issue though for know as this should be done when refstructuring the preferences tab #382. |
Is this still up for grabs? I would give it a try. I would try it to design like @stefan-kolb mentioned |
Yes, this is still open. It would be really nice to have it integrated in the preferences window as a tab and to have it searchable |
Good opportunity to learn and apply JavaFX. We welcome your contribution!
Happy #hacktoberfest!
Manuel <notifications@github.com> schrieb am Mi., 17. Okt. 2018, 10:50:
… Is this still up for grabs? I would give it a try.
Never contributed to your project, though. If I cant make it, I'll write
here again.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#1019 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABTaftfnsFwpAnTls3OGvmULHeWUFyJyks5ulu9rgaJpZM4H2AlI>
.
|
You might have a look at how the search is implemented in https://github.com/dlemmermann/PreferencesFX . |
Hi, we are two students looking for open source project to participate in and found this issue through the awesome beginners list. Or is @WtfJoke still working on this? |
@paulKra I started working on it. Since I am limited on time, I am still in the beginning but still want to do it, hope you guys dont mind :) |
Could you three join forces? This task is not that small and it would be a
pity if work was stopped in the half... If @paulKra would start working and
creating a WIP pull request, @WtfJoke could step in? Or you could have a
sync call? 😇
Manuel <notifications@github.com> schrieb am Fr., 26. Okt. 2018, 15:55:
… @paulKra <https://github.com/paulKra> I started working on it. Since I am
limited on time, I am still in the beginning but still want to do it, hope
you guys dont mind :)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#1019 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABTafixhbC1TsyhxcYP2TTHspEESnnCqks5uoxRHgaJpZM4H2AlI>
.
|
I dont mind doing this together. Im up for a chat/call with @paulKra and your friend. |
I would like to work on implementing this, unless it's still being worked on by someone? |
There is a PR from someone who started working on it,but not finished.
Maybe you can ask the authors or reuse some commits.
There is also another issue from me which is about the window showing all
preferences and searching in there.
That would be awesome if you could implement it
Yash Kothari <notifications@github.com> schrieb am Fr., 8. März 2019, 20:33:
… I would like to work on implementing this, unless it's still being worked
on by someone?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#1019 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AATi5K0ypwD6p7g436OvdXy72uu91Mvuks5vUrsbgaJpZM4H2AlI>
.
|
I didnt forget about the issue/pr (see #4415) but still havent found time to progress. 🙈 @CaptainDaVinci you can continue on my branch. I invited you to my repo. I did some basics and the remaining part is about marking the found entries |
* Feature: implement search filter in show preferences Add a search box in show preferences window to allow users to filter preference options. Resolves #1019 * Add Locale.ROOT when converting to lower case * Fixed indentation for fxml * Added Locale.ROOT when converting to lower case * Using FilteredList to filter options on search * set the count label text, which was accidently removed * Moved checkbox to the top * Bind count label to table size property
How is this related now to #4759? |
The show all preferences window (the table view). is now searchable . The preferences window itself not yet. |
Thanks to @WtfJoke and @CaptainDaVinci this is now implemented in the latest development version. Could you please check the build from http://builds.jabref.org/master/. Thanks! Please remember to make a backup of your library before trying-out this version. |
I took the latest master (JabRef-5.0-dev--snapshot--2019-03-28--master--a73167933.jar) and search looks fine for me. I have however one note: I think in terms of user experience, it would be great to switch to any (or the first) view, which contains the term, if there any hits. Not sure if should create a separate issue for that. |
@WtfJoke yes please open a new issue for it. It's a useful suggestion. |
I have implemented the changes suggested by @WtfJoke , is it okay if I push the changes on the same branch where the search was implemented? |
Nice :) Probably you should create a separate branch from the latest master (since the search branch got already merged) |
There is also an issue of buttons not being highlighted. |
It would be nice to have an incremental search field for searching through the Prerferences in the "Show Prefences" window
The text was updated successfully, but these errors were encountered: