Skip to content

Conversation

marwanemad07
Copy link
Contributor

@marwanemad07 marwanemad07 commented Feb 23, 2025

Closes #11938

This PR adds a behavior for navigation among text areas using a tab button instead of inserting a tab character into the text area.

Mandatory checks

  • I own the copyright of the code submitted and I licence it under the MIT license
  • Change in CHANGELOG.md described in a way that is understandable for the average user (if change is visible to the user)
  • 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.

Before Tab

Before Tab

After Tab

After Tab

After Shift + Tab

After Shift + Tab

CHANGELOG.md Outdated
Comment on lines 14 to 17
- We added a new behavior for the Tab key in empty text areas, ensuring that pressing Tab moves the focus to the next field instead of inserting a tab character. [#11938](https://github.com/JabRef/jabref/issues/11938)

### Added

Copy link
Member

Choose a reason for hiding this comment

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

Please fix the changelog, conform to the way other entries are added. There happens to be an extra heading here.
Also refine the line as "Pressing Tab in empty text areas now moves..."

Copy link
Contributor Author

@marwanemad07 marwanemad07 Feb 23, 2025

Choose a reason for hiding this comment

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

@subhramit From what I understand, I need to add this line to the last 'Added' section and refine it so that it starts with "Pressing Tab..." and continues until the end of the line. Please let me know if this is correct.

Copy link
Member

Choose a reason for hiding this comment

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

Yes.

CHANGELOG.md Outdated
- We added a new CLI that supports txt, csv, and console-based output for consistency in BibTeX entries. [#11984](https://github.com/JabRef/jabref/issues/11984)
- We added a new dialog for bibliography consistency check. [#11950](https://github.com/JabRef/jabref/issues/11950)
- We added a feature for copying entries to libraries, available via the context menu, with an option to include cross-references. [#12374](https://github.com/JabRef/jabref/pull/12374)
- Pressing Tab moves the focus to the next field instead of inserting a tab character. [#11938](https://github.com/JabRef/jabref/issues/11938)
Copy link
Member

@subhramit subhramit Feb 24, 2025

Choose a reason for hiding this comment

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

This became vague - think - does it convey the change properly?
In my previous comment, I told to start with "Pressing Tab in empty text areas fields now moves..."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, I will change it to "Pressing Tab in empty text areas of the entry editor now moves the focus to the next field instead of inserting a tab character. #11938", is this ok or need more refining?

Copy link
Member

Choose a reason for hiding this comment

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

"fields", not "areas". The rest is fine.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is the word "Fields" correct? The text fields already work properly, and this PR specifically solves the issue with text areas.

Copy link
Member

Choose a reason for hiding this comment

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

Are you referring to JavaFX textfields vs textareas? A user won't be aware of that. He will just see them as fields. The issue states the same.
Do clarify if you meant something else?

Copy link
Member

@Siedlerchr Siedlerchr Feb 24, 2025

Choose a reason for hiding this comment

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

"field" refers to Bibtex-Field, entry editor field, not the technical term in this context/changelog

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok got it

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.

works

@Siedlerchr Siedlerchr added this pull request to the merge queue Feb 27, 2025
Merged via the queue into JabRef:main with commit 086ced1 Feb 27, 2025
pasteActionHandler.run();
}

// Custom event handler for Tab key presses.
Copy link
Member

Choose a reason for hiding this comment

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

Follow-up: Proper JavaDoc

- For the Citation key generator patterns, we reverted how `[authorsAlpha]` would behave to the original pattern and renamed the LNI-based pattern introduced in V6.0-alpha to `[authorsAlphaLNI]`. [#12499](https://github.com/JabRef/jabref/pull/12499)
- We keep the list of recent files if one files could not be found. [#12517](https://github.com/JabRef/jabref/pull/12517)
- During the import process, the labels indicating individual paragraphs within an abstract returned by PubMed/Medline XML are preserved. [#12527](https://github.com/JabRef/jabref/issues/12527)
- Pressing Tab in empty text fields of the entry editor now moves the focus to the next field instead of inserting a tab character. [#11938](https://github.com/JabRef/jabref/issues/11938)
Copy link
Member

Choose a reason for hiding this comment

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

Follow-up: Replace Tab by <kbd>Tab</kbd>

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.

Tab in empty text area field should focus the next field

4 participants