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

After/tup 271 fp 1665 submod changes (a2cps migration css tweak) #147

Merged
merged 1 commit into from
Jul 6, 2022

Conversation

wesleyboar
Copy link
Member

Overview

Do not unset footer margin that is now never added (as of TACC/Core-CMS#506).

Related

Changes

  • do not unset footer margin (64be2e2)

Testing & Screenshots

See TACC/Core-CMS#506.

CMS migration stylesheet will not add margin, so no need to unset it.

For: https://jira.tacc.utexas.edu/browse/FP-1665
@wesleyboar wesleyboar changed the title fix(a2cps): remove footer margin unset After/tup 271 fp 1665 excess space above footer Jun 28, 2022
@wesleyboar wesleyboar changed the title After/tup 271 fp 1665 excess space above footer After/tup 271 fp 1665 submod changes (a2cps migration css tweak) Jun 28, 2022
@wesleyboar wesleyboar marked this pull request as ready for review June 28, 2022 22:55
@wesleyboar
Copy link
Member Author

Merging based on approval of TACC/Core-CMS#506.

@wesleyboar wesleyboar merged commit 5160307 into main Jul 6, 2022
@wesleyboar wesleyboar deleted the after/tup-271--fp-1665-excess-space-above-footer branch July 6, 2022 03:25
wesleyboar added a commit to TACC/Core-CMS that referenced this pull request Jul 6, 2022
wesleyboar added a commit to TACC/Core-CMS that referenced this pull request Jul 6, 2022
* fix: load fa icons even if no portal (#504)

We do not want FA icons, but there is at least one usage.

Usage: Frontera homepage, "(→) See All News"

* feat(css): drop .o-site for simple sticky footer trick

1. Do not use o-site class in markup and do not import o-site styles.
2. Install core-styles that makes footer sticky in a simpler way.

For: https://jira.tacc.utexas.edu/browse/FP-1665

* fix(css, submod): migration, no add footer margin

FP-1665 will fix end-of-page padding fro old and new sites.

Thus, no need for old solution, which would add excess space.

For: https://jira.tacc.utexas.edu/browse/FP-1665

* fix(core-styles): load latest for fp-1665

For: https://jira.tacc.utexas.edu/browse/FP-1665

* chore(core-styles): v0.6.0-beta.3 (via gitpkg)

Install @tacc/v0.6.0-beta.3 via https://gitpkg.now.sh to test FP-1665.

For: https://jira.tacc.utexas.edu/browse/FP-1665

* fix(core-styles): v0.6.0-beta.3 (via gitpkg) new

* fix(core-styles): v0.6.0-beta.3 (via gitpkg), 3rd

* fix(core-styles): v0.6.0-beta.3 (via npm)

* chore(submod): merged TACC/Core-CMS-Resources#147

TACC/Core-CMS-Resources#147

For: #506
wesleyboar added a commit to TACC/Core-CMS that referenced this pull request Jul 6, 2022
* feat(tup-271): use TACC/tup-ui:/libs/core-styles

* fix(tup-271): simplify PR diff

* feat: install core-styles via npm at v0.6.1-alpha

* docs(README): change all [Core Styles] links

* feat: install [email protected]

* chore(core-styles): install 0.6.0-beta

* fix(core-styles): template sample markup symlinks

* After/tup 271  fp 1663 system spec figure styles (#503)

* fix(core-styles): s-system-specs unset figure css

* fix(core-styles): no figure css (only .figure css)

- Do not load elements/figure.css.
- Load core-styles that deleted elements/figure.css (and moved its styles).

* fix(core-styles): update temporary pacakge url

* chore: inconsequential change to bust docker cache

* Revert "chore: inconsequential change to bust docker cache"

This reverts commit bd61cdc.

* chore(core-styles): v0.6.0-beta.1

* After/tup 271  fp 1666 frontera header hidden (#505)

* chore(core-styles): v0.6.0-beta.2

* fix: fp-1666 no load what v0.6.0-beta.1 deleted

FP-1666 uses v0.6.0-beta.2 which includes v0.6.0-beta.1.

This eases deploy and testing of FP-1663, FP-1666, and FP-1665.

But v0.6.0-beta.1 removed 'elements/figure.css'; build error on load.

Must not merge FP-1666 (v0.6.0-beta.2) before FP-1663 (v0.6.0-beta.1).

* After/tup 271 fp 1665 excess space above footer (#506)

* fix: load fa icons even if no portal (#504)

We do not want FA icons, but there is at least one usage.

Usage: Frontera homepage, "(→) See All News"

* feat(css): drop .o-site for simple sticky footer trick

1. Do not use o-site class in markup and do not import o-site styles.
2. Install core-styles that makes footer sticky in a simpler way.

For: https://jira.tacc.utexas.edu/browse/FP-1665

* fix(css, submod): migration, no add footer margin

FP-1665 will fix end-of-page padding fro old and new sites.

Thus, no need for old solution, which would add excess space.

For: https://jira.tacc.utexas.edu/browse/FP-1665

* fix(core-styles): load latest for fp-1665

For: https://jira.tacc.utexas.edu/browse/FP-1665

* chore(core-styles): v0.6.0-beta.3 (via gitpkg)

Install @tacc/v0.6.0-beta.3 via https://gitpkg.now.sh to test FP-1665.

For: https://jira.tacc.utexas.edu/browse/FP-1665

* fix(core-styles): v0.6.0-beta.3 (via gitpkg) new

* fix(core-styles): v0.6.0-beta.3 (via gitpkg), 3rd

* fix(core-styles): v0.6.0-beta.3 (via npm)

* chore(submod): merged TACC/Core-CMS-Resources#147

TACC/Core-CMS-Resources#147

For: #506

* chore(core-styles): install 0.6.0

* chore(core-styles): v3.7.11

* fix(changelog): typo, 3.7._ to 3.7.11

* fix(protx): wrong path to core-styles theme

* fix(tup): wrong path to core-styles theme
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