[WIP] Added number of items found#5740
Closed
CloudFenrir wants to merge 2 commits into
Closed
Conversation
tobiasdiez
reviewed
Dec 12, 2019
tobiasdiez
left a comment
Member
There was a problem hiding this comment.
Cool! That's a good idea.
Please also add a bit of space after the total item count.
| selectAllNewEntries(); | ||
| } | ||
| } | ||
| totalItems.textProperty().setValue(String.valueOf(entriesListView.getItems().size())); |
Member
There was a problem hiding this comment.
It should be possible to bind the text directly to the size:
totalItems.textProperty().bind(Bindings.size(viewModel.getEntries())), which can then be moved to line 90.
Author
There was a problem hiding this comment.
It actually didn't work as you said, I just pushed some changes that were missing, I hope everything is ok.
koppor
requested changes
Dec 20, 2019
| addToggle.getStyleClass().add("addEntryButton"); | ||
| addToggle.selectedProperty().bindBidirectional(entriesListView.getItemBooleanProperty(entry)); | ||
| addToggle.selectedProperty().addListener((observable, oldValue, newValue) -> { | ||
| if(observable.getValue()){ |
Member
There was a problem hiding this comment.
Please follow our code formatting guidelines. The auto formatter should add some spaces here.
Member
|
Happy new year to everyone! @CloudFenrir May I ask whether you could find some time to finish this contribution? |
b8ef7b7 to
21c6e5e
Compare
Member
|
Sadly, we did not hear from @CloudFenrir anymore. As we like this feature, we decided to adopt this PR. |
5 tasks
koppor
pushed a commit
that referenced
this pull request
Dec 20, 2021
9dff241 Merge pull request #5796 from dhacker29/patch-1 35949f4 Update turabian-fullnote-bibliography-no-ibid.csl ccfb661 Merge pull request #5793 from benjaminmoon/patch-2 520e127 Update frontiers-medical-journals.csl (#5740) ab629ab Update natura-croatica.csl (#5795) 60b2b17 Rename turabian-fullnote-bibliography-no-ibid to turabian-fullnote-bibliography-no-ibid.csl ad4edff Create turabian-fullnote-bibliography-no-ibid d9b956c Update monographs-of-the-palaeontographical-society.csl 3648ae6 Merge pull request #5792 from StianOby/fix-no-page-number-articles e15b2f7 Update norsk-henvisningsstandard-for-rettsvitenskapelige-tekster.csl d8778cf Removed "page" (s. ) term for articles that do not have page numbers. git-subtree-dir: buildres/csl/csl-styles git-subtree-split: 9dff241
calixtus
pushed a commit
that referenced
this pull request
Dec 20, 2021
* Squashed 'buildres/csl/csl-styles/' changes from 60bf7d5..9dff241 9dff241 Merge pull request #5796 from dhacker29/patch-1 35949f4 Update turabian-fullnote-bibliography-no-ibid.csl ccfb661 Merge pull request #5793 from benjaminmoon/patch-2 520e127 Update frontiers-medical-journals.csl (#5740) ab629ab Update natura-croatica.csl (#5795) 60b2b17 Rename turabian-fullnote-bibliography-no-ibid to turabian-fullnote-bibliography-no-ibid.csl ad4edff Create turabian-fullnote-bibliography-no-ibid d9b956c Update monographs-of-the-palaeontographical-society.csl 3648ae6 Merge pull request #5792 from StianOby/fix-no-page-number-articles e15b2f7 Update norsk-henvisningsstandard-for-rettsvitenskapelige-tekster.csl d8778cf Removed "page" (s. ) term for articles that do not have page numbers. git-subtree-dir: buildres/csl/csl-styles git-subtree-split: 9dff241 * Squashed 'buildres/csl/csl-locales/' changes from d5ee85de8e..95ee22b362 95ee22b362 Merge pull request #243 from jagrg/master 50498dce39 Fix pt-BR translation git-subtree-dir: buildres/csl/csl-locales git-subtree-split: 95ee22b36297a83e3346b950bf08728bfb31d664 Co-authored-by: github actions <jabrefmail+webfeedback@gmail.com>
koppor
pushed a commit
that referenced
this pull request
Jan 1, 2022
f78c707 Update Turabian (full note) for 17.1.3.2 Reprint Editions (#5809) 2169ddc Create art-libraries-society-of-north-america-arlisna-reviews.csl (#5791) 94119a6 Create university-of-south-wales-harvard.csl (#5787) 470925c Update fachhochschule-sudwestfalen.csl (#5811) 1a9eff7 Create annals-of-public-and-cooperative-economics.csl (#5747) d7477f2 Merge pull request #5368 from POBrien333/patch-862 113e683 Merge pull request #5812 from citation-style-language/apa-no-initials 62dacff Create universitat-oberta-de-catalunya-apa.csl 046482c Fix Organization Studies title 1213b95 Make oranization-studies dependent c9641f4 Create apa-no-initials.csl 0b4e047 Modify american-journal-of-archaeology.csl (#5719) ddeda4f Create journal-of-economic-impact.csl (#5707) 40c2696 Update gallia.csl (#5782) 5d5927a Create serbian-archives-of-medicine.csl (#5721) 2777954 Merge pull request #5807 from dhacker29/master 77e3fca Update Society of Biblical Literature Full Note for 6.1.6 4479a5a haaga-helia-university-of-applied-sciences-harvard.csl: no date cites need to be unique (#5726) d00e046 Create university-of-roehampton-harvard.csl (#5732) e76da40 Medicinski Razgledi -- fix locale 75b1554 Add original-date extra to proper place as per SBL Handbook 6.2.17-18 (#5725) d43676e Update harvard-bournemouth-university.csl (#5667) 0312a39 Create ABNT NBR 6023:2018. UNESP – Faculdade de Engenharia de Guarati… (#5705) 6585ff9 Create sinergie-italian-journal-of-management.csl (#5776) a7dc678 Create journal-of-global-health.csl (#5775) 02080ce Merge pull request #5800 from citation-style-language/validation-fixes 8b51d85 change s. d. to s.d. in french style for INBO reports (issue #2) (#5634) 1b4ec09 Create bloomsbury-academic.csl (#5741) 1f28d87 Create zeithistorische-forschungen.csl (#5766) bb11942 Remove superfluous 'container' data from wiley-vch-books 06038a7 Remove et-al affixes be96bd8 Remove et-al affixes from bibliotecae-it.csl 03a94e4 Remove more et-al affixes 07d7423 Revert non-et-al changes to universitat-basel-iberomanistik f8e5fd8 Create medicinski-razgledi.csl (#5789) b4df8ee Create dut-harvard.csl (#5762) 40855c4 Fix affixes on et-al b4fe9b0 Merge pull request #5798 from citation-style-language/uris 3dc4aaf URL-encode documentation URLs 408a3a8 fixes citation-style-language/Sheldon#38 42aca5b Update biophysics-and-physicobiology.csl (#5751) 706bea0 Create universidade-do-porto-faculdade-de-engenharia-chicago-pt.csl (#5742) 269ff91 Update universidade-do-porto-faculdade-de-engenharia-chicago.csl (#5797) 0ba51e7 Update el-profesional-de-la-informacion.csl (#5785) dac3baf Create arthropod-systematics-and-phylogeny.csl (#5772) 9dff241 Merge pull request #5796 from dhacker29/patch-1 35949f4 Update turabian-fullnote-bibliography-no-ibid.csl ccfb661 Merge pull request #5793 from benjaminmoon/patch-2 520e127 Update frontiers-medical-journals.csl (#5740) ab629ab Update natura-croatica.csl (#5795) 60b2b17 Rename turabian-fullnote-bibliography-no-ibid to turabian-fullnote-bibliography-no-ibid.csl ad4edff Create turabian-fullnote-bibliography-no-ibid d9b956c Update monographs-of-the-palaeontographical-society.csl 3648ae6 Merge pull request #5792 from StianOby/fix-no-page-number-articles e15b2f7 Update norsk-henvisningsstandard-for-rettsvitenskapelige-tekster.csl d8778cf Removed "page" (s. ) term for articles that do not have page numbers. 409017c add medium field for eBooks to MLA 8th git-subtree-dir: buildres/csl/csl-styles git-subtree-split: f78c707
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I added the a label showing the number of items found. I will continue working on the selected items in another commit.
Example image (look at the bottom):
