Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Resize column error if double click in FluentDataGrid #1582

Closed
f4n0 opened this issue Feb 27, 2024 · 1 comment · Fixed by #1591
Closed

fix: Resize column error if double click in FluentDataGrid #1582

f4n0 opened this issue Feb 27, 2024 · 1 comment · Fixed by #1591
Labels
bug A bug

Comments

@f4n0
Copy link
Contributor

f4n0 commented Feb 27, 2024

🐛 Bug Report

When using a DataGrid with resizable column width, if you double click on the column separator and then try to resize a column, it will continue to resize on hovering the same column

💻 Repro or Code Sample

On Demo site:

  1. Find a datagrid with resizable column
  2. Double click on separator
  3. Start resizing the column
  4. stop clicking
  5. hoover on column

Here's a video

Fluent.UI.Blazor.components.demo.page.-.Work.-.Microsoft_.Edge.2024-02-27.10-03-32.online-video-cutter.com.1.mp4

🤔 Expected Behavior

I was trying to see if it was like excel (double click as best fit)

😯 Current Behavior

See video

💁 Possible Solution

🔦 Context

🌍 Your Environment

Demo site

  • OS & Device: [e.g. MacOS, iOS, Windows, Linux] on [iPhone 7, PC]
  • Browser [e.g. Microsoft Edge, Google Chrome, Apple Safari, Mozilla FireFox] Edge, Chrome, Opera (not Firefox)
  • .NET and Fluent UI Blazor library Version [e.g. 8.0.2 and 4.4.1] as demo
@vnbaaij vnbaaij added the bug A bug label Feb 27, 2024
@vnbaaij
Copy link
Collaborator

vnbaaij commented Feb 27, 2024

It definitely does not work like Excel 😀

The resizing logic of the DataGrid needs an overhaul. It is on our ToDo list...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants