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

test: v3.11.0 candidate #638

Closed
wants to merge 218 commits into from
Closed

test: v3.11.0 candidate #638

wants to merge 218 commits into from

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented May 10, 2023

Overview

A single branch to test all code planned for v3.11.0 but not in main branch.

✅ Closed because I tested and should get this to main the long way, starting with via #605.

Related / Changes

Includes:

Does Not Include:

Required By:

Requires:

Testing

Note
Bold websites deserve the most attention.

Compare pages of:

UI

Skipped. Too many sites. I am testing. Maybe I have a testing session for CMD to approve/reject deploy of each site.

Notes

Known Issues

  1. Heading differences between staging and prod pages; examples:
    Expected since v3.9.0 / Task/fp 1801 guides breadcrumbs headings #549. See v3.9.0 test session "Migration Steps".
  2. Heading differences between automatic and manual portal home pages; examples:
    The differences are from a bug I just fixed in fix(templates): bump home_portal <hN> levels #640.
  3. Accent color changed from blue to purple.
    The new purple accent color came in v3.10.0 via fe3c242. Many of the old sites are currently on v3.8 or v3.9. The color will change back to blue in v3.11+ if TACC_CORE_STYLES_VERSION = 2.
  4. An <hr> under an <h1> is pulled up too close underneath the <h1>; examples:
    The cause is a difference between migrate.v1-v2.css and site.css bottom margin for <h1>.
  5. APCD's table and modal styles conflict with site.css.
    Because APCD used v3.9 + 25 commits of v3.10.0 and custom table CSS and CSS copied from Core-Portal.
  6. Font that is regular weight and italic style is too thick.
    This bug is not fixed until Core-Styles v1.0, fix(fonts): fp-1891 add regular italic, also add black Core-Styles#71.
  7. Breadcrumbs appear on staging guides, but not prod guides; examples:
    Expected since v3.9.0 / Task/fp 1801 guides breadcrumbs headings #549.
  8. Table font has changed.
    Expected since v3.10.0 / b462ea2 / https://github.com/TACC/Core-Styles/releases/tag/v0.11.0.
  9. Table styles have changed.
    Expected since v3.10.0 / b462ea2 / https://github.com/TACC/Core-Styles/releases/tag/v0.11.0. Fixed via snippet on dev.
  10. Accent color changed from blue to brown.
    Ignoring, because it is only on one unused page (System). All other pages are custom styles (home) or CMS v1 styles. Standard accent color has changed from brown to purple to blue. TACC and Core are as they should be. Frontera retains brown, but its use is not standardized.
  11. Heading differences on automatic pages.
    ProTX prod is simply old and wrong. The staging version is accurate.
  12. Content is narrower than before, but only on v3.8.0 websites.
    This is a newly-discovered v3.9.0 bug. (APCD at v3.9.1+, has this bug.)
  13. Hyperlink underlines are more beneath their text than before.
    Expected since v3.10.0 / b462ea2 / https://github.com/TACC/Core-Styles/releases/tag/v0.10.0 because of x-link's text-underline-offset.

wesleyboar and others added 30 commits November 15, 2022 21:40
The previously installed branch was merged to main.
- fix(fonts): fp-1891 add regular italic, also add black (#71)
- feat(objects): warning about still-disabled o-site
- feat(components): icon updates from Core-Portal
- docs(trumps): comment about lifespan of u-empty
- chore(elements): body margin 0
- feat(dist): commit dist for clients relying on it (#70)
- chore: import dependencies in each pattern
In Core-Styles, icon CSS moved: trumps/icon → components cortal.icon.
J.R. and I agree a real CDN makes sense for Core-Styles, long-term.

J.R. doesn't know about this temp solution yet. He'll have better idea.
No need to re-build Core-Styles since we have it's `dist/`.

Related PR: TACC/Core-Styles#70
- Add core styles version setting.
- Load core-styles from pseudo-CDN.
- Use new CSS layer tech to load stylesheets.
@wesleyboar wesleyboar changed the title Task/v3.11.0 candidate test: v3.11.0 candidate May 10, 2023
wesleyboar added a commit to TACC/Core-CMS-Custom that referenced this pull request May 18, 2023
@wesleyboar wesleyboar added the priority ▲▲ Very high priority label May 18, 2023
@wesleyboar wesleyboar closed this May 23, 2023
@wesleyboar
Copy link
Member Author

This PR is closed. It was always a temporary version of #605, just with a shorter name. It is closed as having zero diff with #605's 163b870.

This was referenced May 23, 2023
@wesleyboar wesleyboar deleted the task/v3.11.0-candidate branch November 13, 2023 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority ▲▲ Very high priority proposal Proposal or request for new task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant