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

Only update auto completion suggestions upon major changes #3078

Merged
merged 2 commits into from
Aug 6, 2017

Conversation

tobiasdiez
Copy link
Member

Fixes #2998. Reused logic from recent change to the backup manager to only propagate mayor changes.

  • 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?)
  • If you changed the localization: Did you run gradle localizationUpdate?

@tobiasdiez tobiasdiez added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Aug 6, 2017
@Siedlerchr Siedlerchr changed the title Only update auto completion suggestions upon mayor changes Only update auto completion suggestions upon major changes Aug 6, 2017
Copy link
Member

@lenhard lenhard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested this locally and it works for me. The CoarseChangeFilter is a nice refactoring. Good job!

@tobiasdiez tobiasdiez merged commit 7fb5949 into master Aug 6, 2017
@tobiasdiez tobiasdiez deleted the improveAutoCompletionCoarse branch August 6, 2017 16:21
@iTao8
Copy link

iTao8 commented Aug 7, 2017

@tobiasdiez I didn't see any improvement. The problems are still there.

JabRef 4.0-dev--snapshot--2017-08-06--master--b7743c4d7
Mac OS X 10.11.6 x86_64
Java 1.8.0_144

@lenhard
Copy link
Member

lenhard commented Aug 7, 2017

@iTao8 Please clarify. This is meant to fix #2998 (comment) and from what I see it does.

As Tobias wrote in the issue, there is nothing to be done about author duplicates and so far no one managed to replicate overlapping dropdown box here (I tried as well and couldn't).

@iTao8
Copy link

iTao8 commented Aug 7, 2017

@lenhard 1. Do you mean no one can reproduce the overlapping dropdown box?

  1. I reported another problem in The autocompletion does not work perfectly #2998: the autocompletion dropdown box for the journal field covers the input. When you are typing a journal name in the journal filed, the input was covered by the autocompletion dropdown box, so you can not see any inputing.

  2. I find new problems. I have a .bib for all my publications (20 papers) and my name are all listed as "Wang, Tao". When I input "Wang, Tao" in the search bar and click "enter", the input automatically changed to "Wang, T." and only ONE entry was found.

  3. Another problem. If I input "WangTao" in the search bar, and then I found this is wrong and want to move the cursor and insert a space between "Wang" and "Tao", but the left key and right key do not work.

@lenhard
Copy link
Member

lenhard commented Aug 7, 2017

@iTao8 Thanks for clarifying! Here's my feedback:

  1. Tobias and me have tried and couldn't reproduce it. I am not aware of anyone else trying, but as long as we can't reproduce this, we can't fix it.

  2. Same here, no one could reproduce the overlap in the journal field (I tried just now). Maybe this is all OSX specific? @chochreiner Can you see how these things look on your machine?

  3. This is already tracked in Autocompletion in search field #3068 There are two simple solutions to this right now: 1. Don't hit enter. The main table will be sorted as you type anyway. 2. Instead of enter, hit escape. Then the autocompletion suggestions go away.

  4. This one I can reproduce :) And I agree with you that the left and right keys should work differently. Could you open a new issue for this? @Siedlerchr This is about key bindings (again). Can you have a look?

@tobiasdiez
Copy link
Member Author

@iTao8 Thank you very much for your feedback.
In fact this PR was not meant to fix any of the issues you mention. I only fixed that you don't get "Te" and "Tes" as autocompletion suggestions if you write "Test" in the field (I thought this is what you meant with "autocompletion covers input").

May I ask you to open a new issue of each of the problems you describe separately. I know this is more work on your side, but would help us to keep track of all the issues and not skip one. Thanks!
(Concerning the overlapping dropdown box: this is maybe a MacOS specific problem).

@iTao8
Copy link

iTao8 commented Aug 7, 2017

@lenhard @tobiasdiez Thanks!

The autocompletion dropdown box covers the input, see the screenshot. The dropdown box started at the bottom of the title filed, so all the input letters in the journal field are covered by this box. I think it should be started at the bottom of the journal filed.
28460120-3a96268c-6e43-11e7-8089-d62c815a35d8

@chochreiner
Copy link
Collaborator

I can confirm the overlap on MacOS (latest development build)
screen shot 2017-08-07 at 12 44 55

@matthiasgeiger
Copy link
Member

Also reproducible on Windows if there is not enough space to display the whole list of suggestions (i.e., the suggestions would be placed over the task bar)

@lenhard
Copy link
Member

lenhard commented Aug 7, 2017

@matthiasgeiger Indeed! Maybe we should just make the size of the drop down list shorter by default.

@tobiasdiez
Copy link
Member Author

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.

5 participants