Skip to content

Implement the GUI for auto date grouping#3

Merged
XD319 merged 5 commits into
implmentation-issue-10822from
gui
Oct 23, 2025
Merged

Implement the GUI for auto date grouping#3
XD319 merged 5 commits into
implmentation-issue-10822from
gui

Conversation

@XD319

@XD319 XD319 commented Oct 19, 2025

Copy link
Copy Markdown
Collaborator

Description

Implemented the GUI of Date group functionality as requested in JabRef#10822, allowing users to create automatic groups based on date fields with flexible granularity options.

Changes

GUI Implementation

  • GroupDialog.fxml: Added Date radio button and configuration panel
  • GroupDialogView.java: Added UI bindings for Date group components
  • GroupDialogViewModel.java: Added Date group properties, initialization, and creation logic
  • FieldFactory.java: Added getDateFields() method for date-related fields
  • JabRef_en.properties: Added localization keys

Features

  • Date option in group dialog left pane
  • Field selection dropdown (date, year, urldate, etc.)
  • Grouping options dropdown (Year/Month/Full Date)
  • Default: DATE field with YEAR grouping

@XD319 XD319 requested a review from elliotgnn October 19, 2025 04:07
@XD319 XD319 self-assigned this Oct 19, 2025

@elliotgnn elliotgnn left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Great job, right now there are some issue when right click the group, error will pop up, suggest fix.

import org.jabref.model.groups.AutomaticPersonsGroup;

add import org.jabref.model.groups.AutomaticDateGroup;


add else if (group instanceof AutomaticDateGroup) { // <-- add this branch
return false; // date groups are computed, not manually assigned
}

// KeywordGroup is parent of LastNameGroup, RegexKeywordGroup and WordKeywordGroup

add AutomaticDateGroup _,


add AutomaticDateGroup _,


add AutomaticDateGroup _,

.map(groupParent -> !(groupParent instanceof AutomaticKeywordGroup || groupParent instanceof AutomaticPersonsGroup))

add AutomaticDateGroup _,

This allows the adapting editing of group to automatic date group.
Also please fix the checks other then binaries

@XD319

XD319 commented Oct 22, 2025

Copy link
Copy Markdown
Collaborator Author

Thank you! I have fixed the bugs and checks other than binaries.

@XD319 XD319 requested a review from elliotgnn October 23, 2025 04:12

@elliotgnn elliotgnn left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Great job on fixing tests and bugs! Looks good to me now.

@XD319 XD319 merged commit fafea35 into implmentation-issue-10822 Oct 23, 2025
46 of 48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants