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

Add value selection(like for month) for custom fields #11162

Merged
merged 5 commits into from
Apr 8, 2024

Conversation

azatyamanaev
Copy link
Contributor

@azatyamanaev azatyamanaev commented Apr 8, 2024

Closes #11109

Added editor class for custom fields. This allows to select values from specified content selectors for corresponding field in entry editor tab.

When you add a custom field and set content selectors for it, in entry editor tab you can choose from a list of specified values.

  1. Add a custom field
    1_add_custom_field

  2. Set content selectors
    2_add_content_selectors

  3. Check value selection in entry editor tab

3_check_value_selection

When the added custom field is multiline, standard text area field is used.

  1. Change added custom field to multiline
    4_change_field_to_multiline

  2. Check how it is displayed in entry editor tab
    5_check_selection_for_multiline

Mandatory checks

  • Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

@azatyamanaev azatyamanaev marked this pull request as ready for review April 8, 2024 07:42
@azatyamanaev azatyamanaev changed the title Add editor class for custom fields to enable value selection from a d… Add value selection(like for month) for custom fields Apr 8, 2024
CHANGELOG.md Outdated
@@ -11,6 +11,9 @@ Note that this project **does not** adhere to [Semantic Versioning](https://semv

### Added

- We added a new keyboard shortcut <kbd>ctrl</kbd> + <kbd>,</kbd> to open the preferences. [#11154](https://github.com/JabRef/jabref/pull/11154)
- [azatayamanaev](https://github.com/azatyamanaev) added value selection(like for month) for content selectors in custom entry types. [#11109](https://github.com/JabRef/jabref/issues/11109)
Copy link
Member

Choose a reason for hiding this comment

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

We do not include our names here. Names are already part of the commit and committers will be highlighted at the release blog posts

@@ -112,7 +114,8 @@ private static SuggestionProvider<?> getSuggestionProvider(Field field, Suggesti
// Enrich auto completion by content selector values
try {
return new ContentSelectorSuggestionProvider((SuggestionProvider<String>) suggestionProvider, contentSelectorValues);
} catch (ClassCastException exception) {
} catch (
ClassCastException exception) {
Copy link
Member

Choose a reason for hiding this comment

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

This is unfortuantely an intellih formatting bug, please move this back to the previous line

Copy link
Member

Choose a reason for hiding this comment

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

@azatyamanaev Do you use IntelliJ 2024.1 or something of 2023?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I use 2024.1

@koppor koppor added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Apr 8, 2024
Copy link
Member

@Siedlerchr Siedlerchr left a comment

Choose a reason for hiding this comment

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

lgtm!

@Siedlerchr Siedlerchr added this pull request to the merge queue Apr 8, 2024
Merged via the queue into JabRef:main with commit 6e6523f Apr 8, 2024
21 checks passed
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.

Add a value selection (like for month) for content selectors in custom entry types
3 participants