Skip to content

fix enter key press in textfield#321

Open
vaneck237 wants to merge 1 commit into
openimis:feature/vite-migrationfrom
Y-Note-SAS:OP-3023
Open

fix enter key press in textfield#321
vaneck237 wants to merge 1 commit into
openimis:feature/vite-migrationfrom
Y-Note-SAS:OP-3023

Conversation

@vaneck237
Copy link
Copy Markdown
Contributor

Description

Search not working when tap enter key in textfield

Type of Change

  • Feature
  • Bug fix
  • Chore (Refactor, Docs, CI/CD)
  • Other, please specify

Related Issue(s) / Task(s)

  • Requires [link to github PR], [link to github PR] needs to be merged first before this one
  • Relates to [link to github PR], this needs to be merged before [link to github PR]
  • External reference (e.g., Jira):

Demo

vid-git.mov

Checklist

  • Unit tests added/modified
  • I18n / translation handled

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Copy link
Copy Markdown
Member

@delcroip delcroip left a comment

Choose a reason for hiding this comment

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

i have no idea of that second sonarCloud, by memory state is always use like that

this.fetchEnabled = props.modulesManager.getConf("fe-core", "shouldFetchInitially", true);
}
componentDidMount() {
document.addEventListener("keypress", this.handleEnter);
Copy link
Copy Markdown
Contributor

@zikani03 zikani03 May 26, 2026

Choose a reason for hiding this comment

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

Perhaps this should be handled via a ref on the element rather than being a document/page wide handler??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants