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

ChipView for crossref and groups #6228

Merged
merged 8 commits into from
Sep 1, 2020

Conversation

dextep
Copy link
Contributor

@dextep dextep commented Apr 1, 2020

Tooltip on crossref field.
Improve display of groups.

Closes #6140.

Tooltip on crossref field.
Improve display of groups.
@JabRef JabRef deleted a comment from codecov bot Apr 1, 2020
@koppor
Copy link
Member

koppor commented Apr 1, 2020

Fixes #6140

Some css fixing ChipView field.
@JabRef JabRef deleted a comment from codecov bot Apr 3, 2020
@dextep dextep closed this Apr 8, 2020
@dextep dextep reopened this Apr 8, 2020
@dextep
Copy link
Contributor Author

dextep commented Apr 8, 2020

Should i continue this issuea?

@tobiasdiez
Copy link
Member

Sorry for the late reply. Everyone is a bit busy with life right now.

I pushed a small change that hopefully fixes the indent issue. If there are still a few display problems, I would suggest you finish the feature first and then we have a look at them in the final steps. Also, if there are problems with the chipview (e.g. the indent), then it would be good to report them at https://github.com/jfoenixadmin/JFoenix.

@tobiasdiez tobiasdiez changed the title Resolve issue JabRef#6140 [WIP] ChipView for crossref and groups Apr 10, 2020
@dextep
Copy link
Contributor Author

dextep commented Apr 14, 2020

Can you please create a pull request so that I can see the complete code. Thanks

Your changes look good, but for some reason suggestionProvider is now null (it wasn't null before I hope)

@tobiasdiez can u help me a bit and check what about it coz i dont have idea whats wrong.

@tobiasdiez tobiasdiez marked this pull request as draft April 17, 2020 16:07
@koppor koppor changed the title [WIP] ChipView for crossref and groups ChipView for crossref and groups Apr 23, 2020
@JabRef JabRef deleted a comment Apr 23, 2020
@JabRef JabRef deleted a comment from thebpc Apr 23, 2020
@tobiasdiez tobiasdiez self-assigned this Apr 28, 2020
@tobiasdiez
Copy link
Member

@dextep did you already had some time to investigate this issue further?

@dextep
Copy link
Contributor Author

dextep commented May 13, 2020

@tobiasdiez yes ofc :)

@calixtus
Copy link
Member

What's the status here?

@calixtus calixtus added status: stale status: waiting-for-feedback The submitter or other users need to provide more information about the issue labels Aug 18, 2020
…ew-jfoenix-implementation

� Conflicts:
�	.idea/runConfigurations/JabRef_Main.xml
�	src/main/java/org/jabref/gui/autocompleter/AutoCompleteSuggestionProvider.java
�	src/main/java/org/jabref/gui/autocompleter/SuggestionProvider.java
�	src/main/java/org/jabref/gui/fieldeditors/FieldNameLabel.java
�	src/main/java/org/jabref/gui/fieldeditors/LinkedEntriesEditor.java
@tobiasdiez tobiasdiez added status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers and removed status: stale status: waiting-for-feedback The submitter or other users need to provide more information about the issue labels Sep 1, 2020
@tobiasdiez
Copy link
Member

Thanks @dextep for your good work. I've now continued working on it and finished this PR. The UI is not perfect, but since it's already an improvement over the existing solution this is now ready for review.

@tobiasdiez tobiasdiez marked this pull request as ready for review September 1, 2020 12:10
@calixtus calixtus merged commit 63270ad into JabRef:master Sep 1, 2020
Siedlerchr added a commit that referenced this pull request Sep 2, 2020
…torShutdown

* upstream/master:
  Add connection check to NetworkTab in preferences (#6838)
  Refine IntelliJ howto to enable built-in compiler again (#6794)
  GitBook: [master] one page modified
  GitBook: [master] 3 pages and 3 assets modified
  Add missing }
  Refactor of remaining preference tabs to PreferencesService (#6836)
  ChipView for crossref and groups (#6228)
  Ignore codecov.io at external link check
  Fix linting issues
  Add "m-mauersberger"
  Update journalList.mv
Siedlerchr added a commit that referenced this pull request Sep 2, 2020
* upstream/master:
  Add connection check to NetworkTab in preferences (#6838)
  Refine IntelliJ howto to enable built-in compiler again (#6794)
  GitBook: [master] one page modified
  GitBook: [master] 3 pages and 3 assets modified
  Add missing }
  Refactor of remaining preference tabs to PreferencesService (#6836)
  ChipView for crossref and groups (#6228)
  Ignore codecov.io at external link check
  Fix linting issues
  Add "m-mauersberger"
  Update journalList.mv
koppor pushed a commit that referenced this pull request Nov 15, 2022
7a10e3f Bluebook: Remove small-caps for case short & legislation (#6305)
ca4a92b Merge pull request #6244 from POBrien333/patch-1107
12743ad Create social-science-history.csl (#6233)
f7c1d57 Update american-chemical-society.csl (#6231)
aca6b23 ready: Update oil-shale.csl (#6225)
aadae55 Create pallas.csl (#6204)
cc7d016 Merge pull request #6253 from nschneid/patch-1
77fab39 Merge pull request #6282 from POBrien333/patch-1119
e2668eb Merge pull request #6263 from POBrien333/patch-1113
7f3244d move style to dependent folder
8584993 Create development-growth-differentiation.csl (#6226)
dfe71ff Create biotechnology-and-bioprocess-engineering.csl (#6227)
0d91742 Create sn-computer-science.csl (#6228)
a0d09b4 Create mots.csl (#6205)
47665e5 Update review-of-international-studies.csl
d573b8b Create computer-supported-cooperative-work.csl
cebec0e ACL: check if edition is ordinal before printing the word "edition"
03a0a39 Re-indent CSL styles
3c64906 fix self link
479c061 fix small issues after visual inspection
a356e72 Create gnosis-journal-of-gnostic-studies.csl

git-subtree-dir: buildres/csl/csl-styles
git-subtree-split: 7a10e3f
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.

Improve display of groups and keywords in entry editor
4 participants