Skip to content

apply spellCheck configuration to input model#4057

Merged
tdonohue merged 3 commits intoDSpace:mainfrom
saschaszott:saschaszott-patch-11
Apr 30, 2025
Merged

apply spellCheck configuration to input model#4057
tdonohue merged 3 commits intoDSpace:mainfrom
saschaszott:saschaszott-patch-11

Conversation

@saschaszott
Copy link
Contributor

Description

With this PR, the spellCheck attribute is also applied to simple input fields based on the application configuration form.spellCheck. Currently, the spellCheck attribute is only set for textarea fields (see #2003).

@tdonohue tdonohue added improvement 1 APPROVAL pull request only requires a single approval to merge labels Mar 6, 2025
@tdonohue tdonohue moved this to 🙋 Needs Reviewers Assigned in DSpace 9.0 Release Mar 6, 2025
@tdonohue tdonohue added this to the 9.0 milestone Apr 30, 2025
Copy link
Member

@tdonohue tdonohue left a comment

Choose a reason for hiding this comment

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

👍 Thanks @saschaszott ! Stumbled on this today, and it's a tiny useful improvement. I tested it and verified that input boxes (onebox) now have spell check capabilities in the submission form.

@github-project-automation github-project-automation bot moved this from 🙋 Needs Reviewers Assigned to 👍 Reviewer Approved in DSpace 9.0 Release Apr 30, 2025
@tdonohue tdonohue merged commit a41a244 into DSpace:main Apr 30, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from 👍 Reviewer Approved to ✅ Done in DSpace 9.0 Release Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge improvement

Projects

No open projects
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants