Skip to content

fix(cms): hide blog list back link until Core-CMS upgrade - #562

Closed
wesleyboar wants to merge 7 commits into
mainfrom
fix/hide-blog-list-back-link
Closed

fix(cms): hide blog list back link until Core-CMS upgrade#562
wesleyboar wants to merge 7 commits into
mainfrom
fix/hide-blog-list-back-link

Conversation

@wesleyboar

@wesleyboar wesleyboar commented Jul 8, 2026

Copy link
Copy Markdown
Member

Overview

Hides the blog list “Back” link on filtered news pages until TUP CMS upgrades to Core-CMS with PORTAL_BLOG_SHOW_BACK_LINK, while keeping the same vertical space below the list.

Important

I could merge this or I could wait until #557 bumps to Core-CMS v4.41.0, then delete this. Regardless, it is here now, to track changes I made live and provide a hash for CDN.

Related

Changes

  • added ad-hoc CSS hide-blog-list-back-link.css (visibility: hidden on .blog-list .blog-back)
  • updated css-ad-hoc-styles.html to load it from jsDelivr

Testing

  1. Open a filtered news list (author, archive, or tag).
  2. Confirm the “Back” link is not visible and spacing below the list is preserved.

UI

keep extra space, but hide text

Notes

Remove the ad-hoc CSS and its <link> after TUP CMS deploys Core-CMS that includes the #1201 template behavior.

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.

Add ad-hoc CSS so filtered news list pages hide the back link before TUP ships Core-CMS with PORTAL_BLOG_SHOW_BACK_LINK.
wesleyboar added a commit to TACC/Core-CMS that referenced this pull request Jul 8, 2026
## Overview

Hides the blog list “Back” link by default on author, archive, and tag
views, with an opt-in portal setting for sites that want it.

## Related

- better way to do TACC/tup-ui#562

## Changes

- **added** `PORTAL_BLOG_SHOW_BACK_LINK` (default `False`)
- **updated** `post_list.html` to render the back link only when the
setting is true
- **updated** `settings_custom.example.py` with the new setting

## Testing

1. Set `PORTAL_BLOG_SHOW_BACK_LINK = False` (default) and open an
author, archive, or tag news list URL.
2. Confirm the “Back” link is absent.
3. Set `PORTAL_BLOG_SHOW_BACK_LINK = True` and repeat; confirm the link
appears.

## UI


https://github.com/user-attachments/assets/15230b56-6057-4381-9ddd-aa06c6531ed9
Added a TODO comment regarding the deletion of a stylesheet link when Core-CMS reaches version 4.41.0.
Use visibility:hidden instead of display:none so filtered news lists keep spacing above the footer.
wesleyboar added a commit to TACC/Core-CMS that referenced this pull request Jul 8, 2026
## Overview

Filtered news list pages lost vertical space below the list after #1200
because the back link was omitted when `PORTAL_BLOG_SHOW_BACK_LINK` is
false. The markup stays in place and is hidden with `visibility: hidden`
so layout matches the previous back link spacing.

## Related

- fixes #1200
- mimicked by TACC/tup-ui#562

## Changes

- **updated** `post_list.html` to always render `.blog-back` on author,
archive, and tag views when the setting is false

## Testing

1. With default `PORTAL_BLOG_SHOW_BACK_LINK`, open a filtered news list
URL.
2. Confirm the “Back” link is not visible and spacing below the list is
preserved.
3. Set `PORTAL_BLOG_SHOW_BACK_LINK = True` and confirm the link is
visible.

## UI

<img width="900" height="475" alt="keep extra space, but hide text"
src="https://github.com/user-attachments/assets/e2b09405-6391-4623-9db1-f428b60f05e7"
/>
@wesleyboar
wesleyboar marked this pull request as draft July 13, 2026 20:44
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.
@wesleyboar

Copy link
Copy Markdown
Member Author

Closing because zero changes since 9dcb3ed because TACC/Core-CMS-Custom#564.

@wesleyboar wesleyboar closed this Aug 3, 2026
wesleyboar added a commit to TACC/Core-CMS-Custom 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
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