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]: the height issue for resizable tables #3875

Merged
merged 2 commits into from
Nov 2, 2023

Conversation

Rajat-Dabade
Copy link
Contributor

@Rajat-Dabade Rajat-Dabade commented Nov 2, 2023

Why this issue

The presence of this issue can be attributed to the introduction of an additional

element with the class name "query-table" at the top of the "ResizableTable" component. This newly added
occupied its own height, which led to the parent container's height not being properly inherited, consequently rendering it non-scrollable.

Solution:

To resolve this issue, we can apply the CSS property height: inherit to the <div>, which will cause it to assume the height of its parent container, restoring its behavior to its previous state.

Copy link

request-info bot commented Nov 2, 2023

We would appreciate it if you could provide us with more info about this issue/pr!

Copy link

github-actions bot commented Nov 2, 2023

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@github-actions github-actions bot added the bug Something isn't working label Nov 2, 2023
Copy link

github-actions bot commented Nov 2, 2023

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@ankitnayan ankitnayan self-requested a review November 2, 2023 13:10
@Rajat-Dabade Rajat-Dabade merged commit b5654c8 into develop Nov 2, 2023
11 checks passed
@prashant-shahi prashant-shahi deleted the fix-query-table-height branch November 2, 2023 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working request-more-info
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants