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: crud2中列宽调整不生效和开启列宽会始终出现横向滚动条的问题 #8738

Closed

Conversation

52flutter
Copy link
Contributor

@52flutter 52flutter commented Nov 14, 2023

What

🤖 Generated by Copilot at 4fc97e9

This pull request fixes a bug and improves the user experience of the Table component in amis-ui. It adjusts the position of the resize handle for the last column header and adds a fallback value for the column width when resizing.

🤖 Generated by Copilot at 4fc97e9

resizeWidth set
table columns adjust smoothly
autumn leaves falling

Why

How

🤖 Generated by Copilot at 4fc97e9

  • Fix a bug where the resizable handle for the last column header in a table would be hidden by the scrollbar when the table is scrollable horizontally (link)
  • Add a fallback value for the resizeWidth variable in the Table component to handle the case where the column width is not explicitly set by the user, but calculated by the table based on the content (link)

@github-actions github-actions bot added the fix label Nov 14, 2023
Copy link

👍 Thanks for this!
🏷 I have applied any labels matching special text in your issue.

Please review the labels and make any necessary changes.

@2betop
Copy link
Collaborator

2betop commented Nov 15, 2023

已包含在 #8679 里面

@2betop 2betop marked this pull request as draft November 15, 2023 06:01
@52flutter
Copy link
Contributor Author

还有第二处修改吧 拖拽不生效

@2betop
Copy link
Collaborator

2betop commented Nov 15, 2023

目前这两处修改我看都有

@2betop 2betop closed this Nov 16, 2023
@52flutter 52flutter deleted the fix-crud2-column-width-resizeable branch November 16, 2023 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants