fix: handle focus button mode columns for reordering

[ugur-vaadin]

Description

This is not to a GridPro edit column issue, but can also be reproduced using regular columns with _focusButtonMode property. When focus button mode is used, the cell is content is wrapped into a div. Because of this, the cell from point is not correctly determined during tracking events.

This PR adds another clause for extracting the cell from point, which enables reordering focus button mode columns.

Fixes #7229

Type of change

• Bugfix
• Feature

Checklist

• I have read the contribution guide: https://vaadin.com/docs/latest/contributing/overview
• I have added a description following the guideline.
• The issue is created in the corresponding repository and I have referenced it.
• I have added tests to ensure my change is effective and works as intended.
• New and existing tests are passing locally with my change.
• I have performed self-review and corrected misspellings.

[vursen]

The original reordering issue appears to have been fixed by #7274. The fact that header cells had focus button divs was a regression. Do we still need this PR? I guess the change that unskips the test is still needed.

[ugur-vaadin]

The original reordering issue appears to have been fixed by #7274. The fact that header cells had focus button divs was a regression. Do we still need this PR? The test can be uncommented though.

I would say it is still relevant because it still fixes the reordering issue when the drop is on the body cells.

[vursen]

I would say it is still relevant because it still fixes the reordering issue when the drop is on the body cells.

Good point, thanks.

[sonarcloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[vaadin-bot]

This ticket/PR has been released with Vaadin 24.4.0.alpha20 and is also targeting the upcoming stable 24.4.0 version.