-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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(tabs): adding tab content to DOM just if selected tab (#1422) #4991
Conversation
ce661fd
to
9f2bcf8
Compare
Codecov Report
@@ Coverage Diff @@
## development #4991 +/- ##
===============================================
- Coverage 74.79% 74.76% -0.03%
===============================================
Files 277 277
Lines 8436 8450 +14
Branches 1603 1605 +2
===============================================
+ Hits 6310 6318 +8
- Misses 1680 1684 +4
- Partials 446 448 +2
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## development #4991 +/- ##
===============================================
- Coverage 74.79% 74.71% -0.09%
===============================================
Files 277 277
Lines 8436 8450 +14
Branches 1603 1605 +2
===============================================
+ Hits 6310 6313 +3
- Misses 1680 1691 +11
Partials 446 446
Continue to review full report at Codecov.
|
9f2bcf8
to
adf694c
Compare
Tabs new demo and previous functionality - tested, looks good. Also will be nice to merge this PR #4946 first |
This change appears in the release note, but seems to be reverted. What is the status of this? Will it be added to the next release? |
PR Checklist
Before creating new PR, please take a look at checklist below to make sure that you've done everything that needs to be done before we can merge it.
Close #1422