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

feat: handled inactive tab handling by removing the display flex override #4708

Merged
merged 2 commits into from
Mar 15, 2024

Conversation

SagarRajput-7
Copy link
Contributor

@SagarRajput-7 SagarRajput-7 commented Mar 15, 2024

Summary

Here antd uses display none property to hide there inactive tabs, but in this case due to a css override - display flex caused by .logs-module-container is overriding this style and prevent the inactive tab content to hide.

Related Issues / PR's

Screenshots

Screen.Recording.2024-03-15.at.12.00.31.PM.mov

Affected Areas and Manually Tested Areas

@CLAassistant
Copy link

CLAassistant commented Mar 15, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ SagarRajput-7
❌ Sagar Rajput


Sagar Rajput seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

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 enhancement New feature or request label Mar 15, 2024
Copy link

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

Copy link

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

@YounixM YounixM merged commit 0365fa5 into develop Mar 15, 2024
10 of 12 checks passed
@prashant-shahi prashant-shahi deleted the pipeline-tabs-issue branch March 15, 2024 07:28
vikrantgupta25 pushed a commit that referenced this pull request Mar 15, 2024
vikrantgupta25 pushed a commit that referenced this pull request Mar 15, 2024
…ride (#4708)

Co-authored-by: Sagar Rajput <[email protected]>
(cherry picked from commit 0365fa5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants