Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

fix(tabs): resolves issue with nested tabs #5719

Closed
wants to merge 1 commit into from
Closed

Conversation

robertmesserle
Copy link
Contributor

Closes #4989

@ThomasBurleson @jelbourn The root cause of this was that I was using find to find the label and body tags, when I actually needed to only look at child elements.


function firstChild (element, tagName) {
var children = element[0].children;
for (var i = 0; i < children.length; i++) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When you check the length property of an HTMLCollection object, it re-checks the DOM, which is slow. You should access the length only once.

@ThomasBurleson ThomasBurleson modified the milestone: 1.0-rc5 Nov 24, 2015
@ThomasBurleson ThomasBurleson added the pr: merge ready This PR is ready for a caretaker to review label Nov 24, 2015
@EladBezalel EladBezalel deleted the issues/4989 branch January 15, 2016 23:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants