Skip to content

chore(cms): delete ad-hoc assets (moved to diff repo) - #571

Merged
jarosenb merged 3 commits into
mainfrom
chore/delete-migrated-ad-hoc-assets
Aug 10, 2026
Merged

chore(cms): delete ad-hoc assets (moved to diff repo)#571
jarosenb merged 3 commits into
mainfrom
chore/delete-migrated-ad-hoc-assets

Conversation

@wesleyboar

@wesleyboar wesleyboar commented Aug 3, 2026

Copy link
Copy Markdown
Member

Overview

Deletes the ad-hoc CSS/JS files that are now versioned in Core-CMS-Custom instead of here.

Related

Changes

  • deleted css/ad-hoc (many files)
  • deleted js/ad-hoc*

Testing

Confirm each deleted file's content matches its counterpart in Core-CMS-Custom's tup/css/ad-hoc/ and tup/js/, except navbar-width-horz-scrollbar.css (matches #535's trimmed version) and c-feed-list-tweaks.css (already deleted independently in #535 — deleting here doesn't conflict).

The actual asset files these snippets link to are now versioned in
Core-CMS-Custom (TACC/Core-CMS-Custom#564, sub-PR of #563) instead of
here. These were reference-only copies (per this repo's "Save Snippets"
convention).

navbar-width-horz-scrollbar.css and c-feed-list-tweaks.css are included
even though tup-ui#535/#325 also touch them: their edits (trim, delete)
are already reflected in Core-CMS-Custom#564.

js/ad-hoc.js and js/ad-hoc/prevent-ugly-urls.js are held back — tup-ui#566
has a pending edit to ad-hoc.js not yet reverted, so deleting it now
would conflict.
Same as this PR's CSS deletions: these are now versioned in
Core-CMS-Custom (TACC/Core-CMS-Custom#564) instead of here.

tup-ui#566 (closed) had a pending edit to ad-hoc.js; it was reverted to
zero diff and closed before this, so no conflict here.
@wesleyboar wesleyboar changed the title chore(cms): delete ad-hoc CSS files migrated to Core-CMS-Custom chore(cms): delete ad-hoc CSS/JS files migrated to Core-CMS-Custom Aug 3, 2026
@wesleyboar wesleyboar changed the title chore(cms): delete ad-hoc CSS/JS files migrated to Core-CMS-Custom chore(cms): delete ad-hoc assets (moved to diff repo) Aug 3, 2026
wesleyboar added a commit that referenced this pull request Aug 3, 2026
c-news.css and c-news--read.css: migrated to Core-CMS-Custom with this
PR's edits composed in (TACC/Core-CMS-Custom#561, merged).

c-feed-list.css, c-feed-list-tweaks.css, and tup-cms.for-core-styles.css:
this PR's deletions/edits to these are already done independently in
tup-ui#570 and #571.
@wesleyboar
wesleyboar force-pushed the chore/delete-migrated-ad-hoc-assets branch from 7154075 to e9c86ec Compare August 4, 2026 16:02
@wesleyboar wesleyboar changed the title chore(cms): delete ad-hoc assets (moved to diff repo) chore(cms): delete ad-hoc assets (moved to diff repo) Aug 4, 2026
@jarosenb
jarosenb merged commit 80812fe into main Aug 10, 2026
1 check passed
@jarosenb
jarosenb deleted the chore/delete-migrated-ad-hoc-assets branch August 10, 2026 18:21
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.

2 participants