Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(dev/tup-cms): django3, blog/news styles + related patterns #581

Merged
merged 308 commits into from
Jul 18, 2023

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Jan 10, 2023

wesleyboar and others added 28 commits December 12, 2022 12:48
- fix(icon): hide text inside icon (TACC/Core-Styles#86)
- fix(package): move npm-watch to devDeps (TACC/Core-Styles@8b2ee16)
- feat(cortal.icons): collapse alias (TACC/Core-Styles#85)
- newest cms button styles
- load cms button in new cms stylesheet
- load core button in old cms stylesheet
- avoid duplicate load of stylesheets
- `core-styles.cms.css` will load core-styles font sizes
    - core-styles@v0-only cuz `assets_core_cms.html` conditional load
- `site.css` will load core-styles font sizes
    - core-styles@v0-only cuz `assets_site.html` conditional load
It was "16px", then moved up to "18px" with font size value changes, but design wants it at "12px".
To start fresh with global Snippet styles, then move em to Core-Styles.
…Core-CMS into task/get-core-styles-beyond-0.11.0
Load core-styles.demo.css before base, just like TACC/Core-Styles.
Let cms styles be applied per pattern, like core-styles does.
- So, core-styles and core-cms demo do not differ.
- Thus, portal and cms can reliably reference demo.

A bonus is that core-styles cms and portal stylesheets are pressured be accurate.
Use new feature from Core-Styles.
Breaking Change: CMS Font sizes and names moved. Check its commit.
By building default context atop core-styles, we define less paths.
This is atop the card & section patterns I've been installing/updating.
@wesleyboar wesleyboar changed the title Dev/tup cms feat: tup cms news, core-styles patterns Jan 10, 2023
@wesleyboar wesleyboar marked this pull request as ready for review July 18, 2023 19:25
@wesleyboar wesleyboar merged commit c464178 into main Jul 18, 2023
@wesleyboar wesleyboar deleted the dev/tup-cms branch July 18, 2023 20:03
@wesleyboar
Copy link
Member Author

The test of #674 on ECEP (which has News and v3.12 bugs since fixed) was the only functional blocker.1

Footnotes

  1. Though the Core-CMS-Resources directory renames introduces a major problem, there is a simple solution that can be performed when they need v3.12. Additionally, the migration process to Core-CMS-Custom includes instructions that document the solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority ▲ High priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant