microsoft / microsoft/fluentui

[Bug]: Datagrid row-focused (OnRowFocus) only fires once

Open
#30,940 6 comments 0 reactions 1 assignee View on GitHub

@chrisdholt is already working on this.

Since Apr 2, 2024.

Resolution: Soft Close web-components
Dominant language
TypeScript
Stars
20.3k
Forks
2.9k
Avg merge
2d 9h
Merged PRs (30d)
46

Description

Library

Web Components (@fluentui/web-components)

System Info
NA
Are you reporting Accessibility issue?

no

Reproduction

https://github.com/ra-one/DataGridMRE

Bug Description

Actual Behavior

The onrowfocus event only fires once

Expected Behavior

The onrowfocus event should fires when row is selected/clicked

Sorry if some details are not available for this bug report, as i I use fluentui-blazor.

I have a data-grid and when i click on a row the event OnRowFocus for blazor and i believe row-focused for web-components should get fired but it only happens on first click, on any subsequent click no events are fired unless i click on different cell in the row, which is not behaviour i think is correct.

Attched video shows the behaviour, I am clicking row-5 multiple times but PreviousId and SelectedId does not change unless I click on date column and same for row-6.

https://github.com/microsoft/fluentui/assets/2285281/541d7945-cb1b-47ee-9315-ae19e93849a8

I have attched blazor code as minimal reproduction example
DataGridMRE.zip

Logs

No response

Requested priority

Blocking

Products/sites affected

No response

Are you willing to submit a PR to fix?

no

Validations
  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • The provided reproduction is a minimal reproducible example of the bug.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.