tree d9f8c35f68c29069932661d72fff7101b9adf578
parent bd5b5718bc173fb8282743f5003e4f13f5a4df4a
author Dirk Pranke <dpranke@google.com> 1635436089 -0700
committer chromium-website-scoped@luci-project-accounts.iam.gserviceaccount.com <chromium-website-scoped@luci-project-accounts.iam.gserviceaccount.com> 1635438038 +0000

Rewrite / fix breadcrumb-handling logic.

The default page template reads the `breadcrumbs` data value from the
front-matter of each page. Previously the export script was generating
a string of HTML to insert directly, but this was buggy and was
effectively hard-coding HTML into the template, rather than letting the
template do the HTML generation.

This CL fixes things by (a) switching the value to be a list of
[href, text] pairs that the template can iterate over, and (b) fixing
the bugs in the string so that the URLs are correct.

In theory we should be able to generate the breadcrumbs during the
build process and not need them to be in the front matter at all, but
making that change will have to wait for now (see crbug.com/1260460).

Bug: 1260459
Change-Id: If5d3ae559914194075a38c614c6ce464e5e08c40
Reviewed-on: https://chromium-review.googlesource.com/c/experimental/website/+/3251600
Reviewed-by: Struan Shrimpton <sshrimp@google.com>
Commit-Queue: Dirk Pranke <dpranke@google.com>