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

Make sure there are tabs before manipulating them.#7688

Open
jpmedley wants to merge 1 commit into
mainfrom
jpmedley-patch-2
Open

Make sure there are tabs before manipulating them.#7688
jpmedley wants to merge 1 commit into
mainfrom
jpmedley-patch-2

Conversation

@jpmedley

Copy link
Copy Markdown
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

chrome-devrel-infra Bot commented Oct 30, 2023

Copy link
Copy Markdown

✅ 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

jpmedley commented Jan 3, 2024

Copy link
Copy Markdown
Contributor Author

@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