Skip to content

feat: update CMS & clean up CMS settings - #557

Draft
wesleyboar wants to merge 7 commits into
mainfrom
feat/core-cms-v4-40
Draft

feat: update CMS & clean up CMS settings#557
wesleyboar wants to merge 7 commits into
mainfrom
feat/core-cms-v4-40

Conversation

@wesleyboar

@wesleyboar wesleyboar commented May 19, 2026

Copy link
Copy Markdown
Member

Overview

  • Update CMS.
  • Clean up CMS settings.

Related

Changes

  1. updated CMS version
  2. deleted …SHOW_ABSTRACT_TAG (renamed in Core, default value matches)
  3. deleted …CUSTOM_MEDIA_POST_CATEGORY (deleted in Core)
  4. added …SHOW_BYLINE (default value is the same)
  5. added SHOW_MAIN_IMAGE (default value is the same)

Testing

  1. Footer links:
    • are white with pale underline
    • not blue with no underline
  2. External articles still open in new tab on different website.
  3. Multimedia article (dev) layout unchanged compared to prod.
  4. The article list and any article show byline (who wrote the article).
  5. Any article's main image is explicitly added to article, not automatically derived from image used for thumbnail.

UI

Notes

Note

Reverted this PR's edit to css-ad-hoc-styles.html — that snippet is now versioned in Core-CMS-Custom and is being deleted from this repo in #569.

- **deleted** `…SHOW_ABSTACT_TAG` (default value is the same)
- **deleted** `…CUSTOM_MEDIA_POST_CATEGORY` (deleted in Core)
- **added** `…SHOW_BYLINE` (default value is the same)
- **added** `SHOW_MAIN_IMAGE` (different than default value)
@wesleyboar wesleyboar changed the title Feat/core cms v4 40 feat: update CMS & clean up CMS settings May 19, 2026
wesleyboar added a commit to TACC/Core-CMS that referenced this pull request May 20, 2026
## Overview

- Implements `PORTAL_BLOG_SHOW_AUTO_MAIN_IMAGE` (default: `False`).
    - Default, `False`, will not render auto main image.
    - The Media placeholder is unaffected.
    - Replaces per-portal custom CSS that hide `.blog-visual`.
- Auto main image:
    - Use `<figure>`.
    - Proper `alt` value.
    - Optional `<figcaption>`.
- Thumbnail image:
    - Use `<figure>`.
    - Proper `alt` value.

## Related

- supersedes TACC/Core-CMS-Custom#536
- documented by TACC/Core-CMS-Custom@c8e5234e
- will test via TACC/Core-Portal-Deployments#191
- required by TACC/tup-ui#557

## Changes

- **added** setting `PORTAL_BLOG_SHOW_AUTO_MAIN_IMAGE`
- **added** conditional rendering in `post_detail.html`
- **change** `<div>` to `<figure>` for news images
- **added** `<figcaption>` for auto main image
- **fixes** bug `alt="None"` in news images

## Testing

> [!NOTE]
> Tested on new
[pprd.cfde-cms.tacc.utexas.edu](https://pprd.cfde-cms.tacc.utexas.edu/news/latest-news/).

| # | Setting | Setup | Expected | Tested? |
| - | - | - | - | - |
| 1 | `False` | Article with a "Main image" set | Main image **absent**
| ⏸️ |
| 2 | `True` | Article with a "Main image" set | Main image
**present**<br>& **No** duplicate image | ⏸️ |

## UI

**Skipped.** I tested **much** more than is listed: images would be
cumbersome to annotate, and without annotation most comparison images
would look very similar; videos would be long and many.

---------

Co-authored-by: Cursor <cursoragent@cursor.com>
wesleyboar added a commit to TACC/Core-CMS-Custom that referenced this pull request Jul 31, 2026
## Overview

Update `tup/html/` with every snippet change from a currently-open
tup-ui PR.

## Related

- requires #559
- migrates snippet changes from:
    - TACC/tup-ui#568
    - TACC/tup-ui#325
    - TACC/tup-ui#535
    - TACC/tup-ui#557
    - TACC/tup-ui#562
    - TACC/tup-ui#565
    - TACC/tup-ui#566

## Changes

- **added** `embed-vislab-*.html`
    <sup>latest versions from:</sup>
    - TACC/tup-ui#568
- **updated** `css-ad-hoc-styles.html`
    <sup>hand-composed from:</sup>
    - TACC/tup-ui#325
    - TACC/tup-ui#535
    - TACC/tup-ui#557
    - TACC/tup-ui#562
    - TACC/tup-ui#565
- **updated** `js-ad-hoc-scripts.html`
    <sup>hand-composed from:</sup>
    - TACC/tup-ui#535
    - TACC/tup-ui#566

## Testing

1. Compare `embed-vislab-*` files against the branch of
TACC/tup-ui#568 to confirm content matches.
2. For `css-ad-hoc-styles.html` and `js-ad-hoc-scripts.html`, compare
against each contributing PR's diff to confirm every intended change
made it in.
That snippet is being deleted from this repo in favor of Core-CMS-Custom
(#569); keep it untouched here so #569 doesn't conflict with this PR.
jarosenb pushed a commit that referenced this pull request Aug 10, 2026
* chore(cms): delete snippets migrated to Core-CMS-Custom

These 9 template copies were reference-only records of djangocms-snippet
content per this repo's "Save Snippets" convention. They now live in
Core-CMS-Custom (TACC/Core-CMS-Custom#559) with no open PR here still
editing them, so the local copies are redundant.

* chore(cms): delete remaining snippets migrated to Core-CMS-Custom

css-ad-hoc-styles.html and js-ad-hoc-scripts.html were held back from the
prior commit because open PRs (#325, #535, #557, #562, #565, #566) still
edited them here. Those PRs are being updated to drop their edits to these
two files in favor of Core-CMS-Custom (TACC/Core-CMS-Custom#560), so the
local copies can go too.

* chore(cms): delete redirect.html and open-external-post.html

Unlike this PR's other deletions, these two were never migrated to
Core-CMS-Custom (they're djangocms-snippet "template"-field templates,
not manually-synced reference copies, per the original migration's
notes). They're deleted here because #535 already deletes them
independently — the external-post-link and redirect behavior they
provide is now handled by Core-CMS itself.
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