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

[two_dimensional_scrollables] TableView can't scroll horizontally if rows fit on screen but columns don't. #147730

Closed
isenbj opened this issue May 2, 2024 · 4 comments
Labels
r: fixed Issue is closed as already fixed in a newer version

Comments

@isenbj
Copy link

isenbj commented May 2, 2024

Steps to reproduce

  1. Open this dartpad https://dartpad.dev/?id=4424936c57ed13093eb389123383e894
  2. Set the maxYIndex to be something that fits fully on the screen so that vertical scrolling is not necessary.
  3. Leave maxXIndex to be something that does not fit the screen so that horizontal scrolling is necessary to see all the columns.
  4. Observe that you cannot drag to see all of the columns.
  5. Observe that you can resize the screen so that the rows fall out of view and you need to vertical scroll to see them. Suddenly horizontal scrolling is enabled as well.

Note: I am using the given dartpad example here to demonstrate the issue. My personal issue is through using the TableView.builder, however the issue is the same.

Also:
I believe this issue was closed prematurely which is my same problem: #135384

The linked issue where it was claimed to be a duplicate is not related.

Expected results

To be able to horizontal scroll when there are more columns than fit on screen, but the rows do fit.

Actual results

Scrolling is not possible so a user cannot see all the columns.

Code sample

Code sample

https://dartpad.dev/?id=4424936c57ed13093eb389123383e894

Screenshots or Video

Screenshots / Video demonstration

[Upload media here]

Screen.Recording.2024-05-02.at.12.24.50.PM.mov

Logs

Logs

Not needed

Flutter Doctor output

Doctor output

Not needed

@darshankawar darshankawar added the in triage Presently being triaged by the triage team label May 3, 2024
@darshankawar
Copy link
Member

Thanks for the report @isenbj
You'll be happy to know that the expected behavior works on latest master version, as shown below:

147730.mov

The one where it doesn't work must be on stable. You may switch to master and re-run your scenario to confirm at your end as well.

@darshankawar darshankawar added the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 3, 2024
@isenbj
Copy link
Author

isenbj commented May 3, 2024

I can confirm it works on master for the example. I can't run master on my app though due to several dependency breaks with removal of different theming options (body1 -> bodyLarge for example). I can just use DiagonalDragBehavior.none rather than .free for now though.

@github-actions github-actions bot removed the waiting for customer response The Flutter team cannot make further progress on this issue until the original reporter responds label May 3, 2024
@darshankawar
Copy link
Member

Thanks for the update. Since it works on master, it is considered to be fixed and per our triage policy, will close as resolved.

@darshankawar darshankawar added r: fixed Issue is closed as already fixed in a newer version and removed in triage Presently being triaged by the triage team labels May 6, 2024
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
r: fixed Issue is closed as already fixed in a newer version
Projects
None yet
Development

No branches or pull requests

2 participants