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

Make sure there are tabs before manipulating them. #7688

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jpmedley
Copy link
Contributor

Addresses #7601

@AmySteam This seems like a reasonable addition to this code sample. You probably wouldn't be trying to run it unless you had tabs open to documentation, but it's also probably easy to overlook. How many hours have we all wasted because silly minor thing we need to do.

This is already in the related sample.

Addresses #7601 

@AmySteam This seems like a reasonable addition to this code sample. You probably wouldn't be trying to run it unless you had tabs open to documentation, but it's also probably easy to overlook. How many hours have we all wasted because silly minor thing we need to do.

This is already in the related sample.
@jpmedley jpmedley requested a review from AmySteam October 30, 2023 15:23
@chrome-devrel-infra
Copy link

chrome-devrel-infra bot commented Oct 30, 2023

✅ Preview (static) for commit 47c9f0b available at https://pr-7688-static-dot-dcc-staging.uc.r.appspot.com/.

The following pages likely changed with this PR:

@jpmedley
Copy link
Contributor Author

jpmedley commented Jan 3, 2024

@AmySteam, Is this something you want to do?

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

Successfully merging this pull request may close these issues.

2 participants