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

Warning: Label 'page load time' already exists for console.time() #208

Closed
gpaciga opened this issue Mar 12, 2021 · 2 comments
Closed

Warning: Label 'page load time' already exists for console.time() #208

gpaciga opened this issue Mar 12, 2021 · 2 comments
Assignees

Comments

@gpaciga
Copy link

gpaciga commented Mar 12, 2021

When using timers, an warning is reported:

Warning: Label 'page load time' already exists for console.time()

It looks like this might just be because the timer is set twice in packages/cli/src/lib/axe-test-urls.ts?

    if (config.timer) {
      events?.startTimer('page load time');
    }

Are these meant to be two different timers or is this just an error?

In either case, it is probably wise to make the name more specific, (e.g. axe page load time) to avoid the potential for conflicts with non-axe timers (which is what I thought this was originally from).

@padmavemulapati
Copy link

padmavemulapati commented May 17, 2021

seeing the difference in the axe-test-ulrs.ts file - instead of page load time , updated as axe page load time so that it is looking for the time to load only for the test page after triggering axe
image

@michael-siek
Copy link
Member

Closing no docs required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants