Skip to content

fix(search): pin google-search.css version with fixes - #554

Merged
wesleyboar merged 8 commits into
mainfrom
ai/google-search-styles-migration-f17f90
Jul 31, 2026
Merged

fix(search): pin google-search.css version with fixes#554
wesleyboar merged 8 commits into
mainfrom
ai/google-search-styles-migration-f17f90

Conversation

@wesleyboar

@wesleyboar wesleyboar commented Jul 31, 2026

Copy link
Copy Markdown
Member

Overview

  1. Use Google Search styles now in this repo.
  2. Point to latest copy of the styles (includes fixes).

Related

Changes

  • changed URL for google-search.css

Testing

  1. Update a project to use new snippet markup.
    (e.g. artscivis, cipp, craftlab, ctrn, wtcs)
  2. Visit a project's search page at a wide viewport (e.g. 1920px).
  3. Verify:
    1. gray search results header bkgd has no gaps
    2. search results are centered

wesleyboar and others added 4 commits July 31, 2026 11:34
… fix

Bumps the pinned Core-CMS commit for google-search.css from 8b59bd38
to e5b17086, migrating in the gray header background gap fix.

Related: TACC/Core-CMS#1218, #553

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
….css

Repoints the artscivis, cipp, craftlab, ctrn, and wtcs search snippets
from Core-CMS's google-search.css to Core-CMS-Custom's own copy
(pinned to the migration branch commit), completing the migration
away from Core-CMS. Reverts the prior generic_assets change, since
that file's content is now owned by the migration in #555.

Related: #555

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
#555 merged, so its inlined google-search.css now lives on main.
Re-pins the artscivis, cipp, craftlab, ctrn, and wtcs search snippets
from the migration branch commit to that merge commit.

Related: #555

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@wesleyboar wesleyboar changed the title fix(search): pin google-search.css to Core-CMS commit with header-gap fix fix(search): pin google-search.css version with fixes Jul 31, 2026
@wesleyboar
wesleyboar marked this pull request as draft July 31, 2026 16:54
Pins to the v1.5 minor line instead of a specific commit, so future
v1.5.x patch releases apply automatically without re-pinning.
Widens the pin from v1.5 to the v1 major line, so it tracks any
future v1.x.x release automatically.
@wesleyboar
wesleyboar marked this pull request as ready for review July 31, 2026 17:42
@wesleyboar
wesleyboar merged commit 13b799b into main Jul 31, 2026
@wesleyboar
wesleyboar deleted the ai/google-search-styles-migration-f17f90 branch July 31, 2026 17:42
wesleyboar added a commit that referenced this pull request Jul 31, 2026
## Overview

Reverts LCCF's one-off search snippet back to the standard pattern used
by every other project.

## Related

- reverts #553
- according to #554

## Changes

- **reverted** LCCF's search snippet to the shared markup/stylesheet
pattern (artscivis, cipp, craftlab, ctrn, wtcs)

## Testing

1. Update LCCF's search snippet in the CMS admin interface to this
file's content
2. Visit LCCF's search page
3. Verify search results render and are styled consistently with the
other projects
wesleyboar added a commit that referenced this pull request Jul 31, 2026
## Overview

Repoints the iframe script/stylesheet from a specific commit to a
version range, matching the search stylesheet's pinning approach.

## Related

- per [#554](#554)

## Changes

- **pinned** iframe script/stylesheet to the `v1.5` semver range in
`generic_assets/html/iframe-styles.html`,
`iframe-styles-and-auto-height.html`, `iframe-tacc-docs.html`, and
`ctrn_assets/snippets/iframe-styles.html`

## Testing

1. Visit a project's page that embeds an iframe via one of these
snippets
2. Verify the iframe still renders with correct styles and auto-height
behavior
wesleyboar added a commit that referenced this pull request Aug 3, 2026
## Overview

Use latest Google Search styles for TUP.

## Related

- #555
- #557
- #554

## Changes

- **updated** snippet markup

## Testing

1. [TACC search](https://tacc.utexas.edu/search/?q=test) renders like
[Texas CTRN search](https://texasctrn.org/search/?q=test).
1. [Snippet
#89](https://tacc.utexas.edu/admin/djangocms_snippet/snippet/89/change/)
content matches this diff.

## UI

Skipped.
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

Successfully merging this pull request may close these issues.

1 participant