Add missing target="_blank" to links in 221 designer bios#10226
Add missing target="_blank" to links in 221 designer bios#10226felipesanches merged 1 commit intomainfrom
Conversation
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
|
I consider this to be objectively good and uncontroversial. |
|
@nyshadhr9 Will the backend recut all the font instances if we merge this? We're only touching the html files but I remember updating the metadata.pb files will trigger it. |
Automated fix: added target="_blank" attribute to all <a> tags in bio.html files that were missing it. This ensures external links open in a new tab, consistent with the editorial guide. Assisted by an AI agent (Claude Opus 4.6)
|
@nyshadhr9 can we merge this? |
|
@nyshadhr9, I hope it is fine to merge this ;-) |
65c082d to
0153186
Compare
|
Note: This post was generated by an AI agent (Claude) working under the guidance of @felipesanches, but submitted without human review. @felipesanches himself would still need to participate in the PR thread if he wants to contribute to the review. Fixed the commit attribution: replaced |
|
Sorry for the late reply. Yes, this can be merged. It doesn't lead to re-cutting instances. |
|
thanks! |
WARNING: This is an ongoing experiment to evaluate the quality of work performed by AI agents on behalf of (and under the guidance of) Felipe Sanches. These contributions have NOT been reviewed by any human prior to submission. Felipe himself would need to participate in the PR thread to contribute a review. Please treat all content as unverified and requiring human fact-checking.
Summary
target="_blank"attribute to all<a>tags in designerbio.htmlfiles that were missing ittarget="_blank"attribute was added to existing linksContext
An automated audit of all 364 designer bios against the Editorial Guide found that missing
target="_blank"was the single most common issue, affecting 60% of bios. This PR addresses that issue across the entire catalog in one pass.Verification
target="_blank"attributes were introducedtarget="_blank"on all links were left untouchedtarget="_blank"attribute was added — no other content was modifiedTest plan
target="_blank"was addedtarget="_blank"in any<a>tag🤖 Generated with Claude Code