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

fix(core-styles): fp-1723 fix o-section--banner on safari #29

Merged

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Jul 12, 2022

Overview / Changes

  1. Replace tricky CSS with simple, obvious CSS.
  2. Adapt .o-section--banner selectors to new markup.

I changed the markup slightly in dev and pprd.1

Related

Testing / UI

See TACC/Core-CMS#513.

Notes

Released in @tacc/core-styles@0.7.0-beta via core-styles/v0.7.0-beta.

Footnotes

  1. For details, see https://github.com/TACC/Core-CMS/pull/513.

@wesleyboar wesleyboar changed the title fix(o-section): fp-1723 drop trick, use easier css fix(core-styles): fp-1723 no css trick + new html Jul 13, 2022
@wesleyboar wesleyboar changed the title fix(core-styles): fp-1723 no css trick + new html fix(core-styles): fp-1723 tweak o-section--banner Jul 13, 2022
@wesleyboar wesleyboar changed the base branch from main to core-styles/v0.7.0 July 13, 2022 16:41
@wesleyboar wesleyboar changed the title fix(core-styles): fp-1723 tweak o-section--banner fix(core-styles): fp-1723 fix o-section--banner on safari Jul 13, 2022
@wesleyboar
Copy link
Member Author

Merged based on approval of dependent TACC/Core-CMS#513.

@wesleyboar wesleyboar merged commit addafe9 into core-styles/v0.7.0 Jul 13, 2022
@wesleyboar wesleyboar deleted the bug/fp-1723-frontera-header-hidden-in-safari branch July 13, 2022 20:27
@wesleyboar wesleyboar mentioned this pull request Jul 13, 2022
wesleyboar added a commit that referenced this pull request Jul 15, 2022
* fix(core-styles): fp-1723 fix o-section--banner on safari (#29)

* fix(o-section): fp-1723 drop trick, use easier css

* fix(o-section): fp-1723 adapt to new markup

* feat(core-styles): tup-308, new global color (#25)

* Core styles/v0.7.0 beta (#31)

* Squashed commit of the following:

commit 81323f7
Author: Wesley Bomar <wbomar@tacc.utexas.edu>
Date:   Wed Jul 6 15:08:43 2022 -0500

    feat(core-styles): tup-308, new global color

* Squashed commit of the following:

commit 4c21f96
Author: Wesley Bomar <wbomar@tacc.utexas.edu>
Date:   Wed Jul 13 10:26:48 2022 -0500

    fix(o-section): fp-1723 adapt to new markup

commit 31077dc
Author: Wesley Bomar <wbomar@tacc.utexas.edu>
Date:   Tue Jul 12 17:39:33 2022 -0500

    fix(o-section): fp-1723 drop trick, use easier css

* chore(core-styles): v0.7.0-beta

* chore(core-styles): v0.7.0-beta .npmrc

* chore(core-styles): v0.7.0-beta package…json

* feat(core-styles): add git tagging to publish steps in readme

* chore(core-styles): v0.7.0

* chore(core-styles): nx format:write
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