feat(tup): sync snippets to latest open tup-ui PRs - #560
Merged
Conversation
wesleyboar
force-pushed
the
ai/tup-ui-snippets-sync-latest
branch
from
July 31, 2026 19:17
33fd6bc to
2f9935f
Compare
wesleyboar
force-pushed
the
ai/tup-ui-snippets-sync-latest
branch
from
July 31, 2026 20:20
41fd6ee to
3e1e76d
Compare
wesleyboar
added a commit
that referenced
this pull request
Jul 31, 2026
## Overview
Adds a new `tup` project for the main TACC site (tup-ui) and migrates
its manually-synced djangocms-snippet copies here for version control.
## Related
- tup-ui's `apps/tup-cms/src/taccsite_cms/templates/snippets/` folder
## Changes
- **added** `tup/html/` with 11 snippet files copied as-is from tup-ui's
`main` branch
- **updated** project list in root `README.md` to include TACC
## Testing
1. Compare each new file's content against its `git log` history in
tup-ui to confirm it matches the last "save" commit.
## Notes
- Not migrated: `css-js-staff-profile.html` and `redirect.html` and
`news/open-external-post.html` remain in tup-ui — they are loaded
directly via a snippet's "template" field (confirmed via `{{html}}`
context-variable usage and an explicit doc comment for the first).
- Not migrated (not a snippet): `a-legend-key-to-the-icons.html` and
`css-js-staff-profile.md` are reference/documentation files, not
djangocms-snippet content.
- Follow-up PR #560 planned to bring these files current with open
tup-ui PRs that update snippets.
- tup-ui's copies are intentionally left in place for now; deleting them
is planned as a separate future PR.
tup-ui#568 (open, draft) adds 3 more manually-synced snippets not yet on tup-ui main: embed-vislab-cal-content.html, embed-vislab-cal.html, and embed-vislab-form.html. Its 4th new file, iframe-styles-and-iframe-auto-height.html, is excluded here because it is a CDN-pin loader (like css-ad-hoc-styles.html/js-ad-hoc-scripts.html) loaded directly via a snippet's "template" field, not manually synced. No other open tup-ui PR changes any already-migrated snippet. The other snippet-touching open PRs (tup-ui#558, #566, #565, #562, #557, synced, so nothing else here needs updating yet.
…om all open PRs Hand-composed via sequential 3-way merges (git merge-file) against the shared base blob, since none of the tup-ui PRs touching these files are stale: css-ad-hoc-styles.html layers tup-ui#325, #535, #557, #562, #565. #325 and #535 both drop the same c-feed-list-tweaks.css line (merged cleanly as one change). #557 drops the trailing figcaption-opacity TODO block. #562 and #535 each add an independent TODO'd <link> in the same spot; kept both, older PR (#535) first. #565 bumps the Core-CMS-Custom pin and adds core-styles-fixes.css; its version of that line wins since no other open PR touches it. js-ad-hoc-scripts.html layers tup-ui#535 (adds sekizai_tags load, drops the stale "delete js-prevent-ugly-urls" comment) and #566 (bumps the ad-hoc.js pin). Kept #566's bumped pin with #535's comment removal, since #566 never touched that comment either way.
wesleyboar
force-pushed
the
ai/tup-ui-snippets-sync-latest
branch
from
July 31, 2026 20:23
3e1e76d to
08586bd
Compare
wesleyboar
commented
Jul 31, 2026
Comment on lines
+38
to
+48
| {# To fix tab layout on news article #} | ||
| {# TODO: Integrate as fix into Core-CMS (use `:not` to limit its selector) #} | ||
| {# SEE: https://tacc.utexas.edu/news/latest-news/2030/06/30/advanced-news-template/#:~:text=Tabs #} | ||
| <style> | ||
| :is(article.post-detail) .blog-content li + li, | ||
| :is(article.post-detail) .blog-lead li + li { | ||
| &:is(.nav-tabs li) { | ||
| margin-top: unset; | ||
| } | ||
| } | ||
| </style> |
Member
Author
There was a problem hiding this comment.
This came from live snippet. No tup-ui PR was open to track it.
This was referenced Jul 31, 2026
wesleyboar
added a commit
that referenced
this pull request
Aug 3, 2026
## Overview Migrates CSS/JS assets (that `tup/html/css-ad-hoc-styles.html` and `js-ad-hoc-scripts.html` load) from [TACC/tup-ui] to here. <details> [TACC/tup-ui]: https://github.com/TACC/ The snippets files were already migrated (#559, #560); this closes the loop so those snippets' `<link>`/`<script>` tags can point at this repo's CDN pins instead of tup-ui's. </details> ## Related - adds to #563 - integrates changes from: - TACC/tup-ui#566 - TACC/tup-ui#535 - TACC/tup-ui#562 > [!IMPORTANT] > **Follow-up:** Repoint `tup/html/css-ad-hoc-styles.html` and `js-ad-hoc-scripts.html`'s CDN URLs at this repo (not done in this PR). ## Changes - **added** `tup/css/ad-hoc/` (many files) - **added** `tup/js/ad-hoc.js` - **added** `tup/js/ad-hoc/manage-participants-field-required-attr.js` - **added** `tup/css/ad-hoc/hide-blog-list-back-link.css` (from TACC/tup-ui#562, open) - **added** `tup/css/ad-hoc/adjust-logo.css` (from TACC/tup-ui#535, open) - **updated** `navbar-width-horz-scrollbar.css` ## Testing 1. Confirm each file's content matches TACC/tup-ui's current copy exactly **except**: - `navbar-width-horz-scrollbar.css` matches TACC/tup-ui#535's branch - `adjust-logo.css` matches TACC/tup-ui#535's branch (doesn't exist on tup-ui `main`) - `hide-blog-list-back-link.css` matches TACC/tup-ui#562's branch (doesn't exist on tup-ui `main`) - `ad-hoc.js`/`manage-participants-field-required-attr.js` matches the commit already pinned in the live snippet, TACC/tup-ui@0c0beee5
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview
Update
tup/html/with every snippet change from a currently-open tup-ui PR.Related
Changes
embed-vislab-*.htmllatest versions from:
css-ad-hoc-styles.htmlcomposed from:
plus a tab-layout fix that only exists in the live CMS admin snippet, not in any tup-ui branch; removals from PRs above that haven't deployed yet (
c-feed-list-tweaks.css, the figcaption-opacity block) are kept with a TODO instead of deleted outrightjs-ad-hoc-scripts.htmlhand-composed from:
Testing
embed-vislab-*files against the branch of feat: vislab form & cal - snippets tup-ui#568 to confirm content matches.css-ad-hoc-styles.htmlandjs-ad-hoc-scripts.html, compare against each contributing PR's diff and the live CMS admin snippet to confirm every intended change made it in and nothing admin-only got missed.