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

Grand unified preferences dialog #7384

Merged
merged 24 commits into from
Feb 1, 2021
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
29920df
Introduced SimplePreferencesDialog and converted CustomizeGeneralFiel…
calixtus Jan 25, 2021
7e83e3f
Moved preferences tabs to their own packages
calixtus Jan 25, 2021
00a13b8
Removed SetupGeneralFieldsAction
calixtus Jan 25, 2021
0477242
Converted ImportCustomizationDialog
calixtus Jan 25, 2021
a038042
Converted ExportCustomizationDialog
calixtus Jan 25, 2021
57c49e1
Cleanups and l10m
calixtus Jan 25, 2021
a4f29dd
Checkstyle
calixtus Jan 25, 2021
1fda9c3
Cleanups
calixtus Jan 25, 2021
ffeb239
Converted ManageJournalAbbreviations
calixtus Jan 26, 2021
30b435d
l10n
calixtus Jan 26, 2021
d42670d
Converted KeyBindingsDialog
calixtus Jan 26, 2021
0af42f4
l10n
calixtus Jan 26, 2021
280aed7
Fixed checkstyle and tests
calixtus Jan 26, 2021
97a1146
Converted ManageProtectedTermsDialog
calixtus Jan 27, 2021
ee19398
Simplified call to CitationKeyPatternPanel in markup
calixtus Jan 29, 2021
fd0f821
Removed hardwired width settings
calixtus Jan 29, 2021
784eb5d
Renamed files for consistent naming scheme
calixtus Jan 29, 2021
0d4205d
Fixed dependency injection and renamed variables for consistency
calixtus Jan 29, 2021
2e186f5
CHANGELOG.md
calixtus Jan 29, 2021
d1ca90b
Final cleanups
calixtus Jan 29, 2021
0c7cd39
Merge remote-tracking branch 'upstream/master' into grupd
calixtus Jan 29, 2021
bdb1560
Fixed simple remarks
calixtus Jan 30, 2021
cc29b5f
Fixed localization tests
calixtus Jan 30, 2021
3823490
Merge remote-tracking branch 'upstream/master' into grupd
calixtus Feb 1, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -152,8 +152,12 @@ public List<String> getBindings() {

@Override
public boolean equals(Object o) {
if (this == o) return true;
if (o == null || getClass() != o.getClass()) return false;
if (this == o) {
return true;
}
if (o == null || getClass() != o.getClass()) {
return false;
}

KeyBindingRepository that = (KeyBindingRepository) o;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ public AbbreviationsFileViewModel(Path filePath) {
this.path = Optional.ofNullable(filePath);
this.name = path.get().toAbsolutePath().toString();
this.isBuiltInList = new SimpleBooleanProperty(false);
this.abbreviations.add(new AbbreviationViewModel(null));
Copy link
Member Author

Choose a reason for hiding this comment

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

This is a so called PseudoAbbreviation. I have not really understood what the real benefit is of this (maybe one can doubleclick an empty space in the List and insert a new abbreviation?) It's buggy, makes the code quite complicated to understand and I see no real benefit in it. ... ... But I put it back in for now, as several tests break, because they expect this PseudoAbbreviation and it would take me now valuable life time I would rather like to spend in finishing this PR, maybe as soon as I have completed everything else...

}

/**
Expand Down