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

SHS-5772 - Regression: External Links on Postcards are wrong color #1591

Merged

Conversation

mariannuar
Copy link
Collaborator

@mariannuar mariannuar commented Aug 7, 2024

READY FOR REVIEW

Summary

Fix regression in external links on Postcards title

Need Review By (Date)

08/09

Urgency

medium

Steps to Test

  1. Go to the following sites
  1. Check that the external links are now lookins as expected (colour is black text with bottom border)
  2. On Colorful and Traditional
  3. Go to /components/collections-cards/collections-postcards
  4. Edit some of them and add internal and external links for all the variations of Postcards (except the vertical linked card)
  5. Check the look the same as expected
  6. Also test everything in multiple colour palettes (pay attention to the Warbler color pairing in Traditional themes)

PR Checklist


@mariannuar mariannuar self-assigned this Aug 7, 2024
@mariannuar mariannuar requested a review from cienvaras August 7, 2024 00:40
Base automatically changed from 11.1.4-release to develop August 7, 2024 15:48
@cienvaras cienvaras changed the base branch from develop to fk-stnfd-sprint-56 August 8, 2024 18:54
Copy link
Collaborator

@cienvaras cienvaras left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! :shipit:

@cienvaras cienvaras merged commit 254bc77 into fk-stnfd-sprint-56 Aug 9, 2024
14 of 15 checks passed
@cienvaras cienvaras deleted the shs-5772-fix-regression-in-external-links branch August 9, 2024 16:18
joegl pushed a commit that referenced this pull request Aug 13, 2024
* SHS-5692: Implementation: New Spotlight design for Colorful and Traditional (#1589)
* SHS-5772: Regression: External Links on Postcards are wrong color (#1591)
* SHS-5629: Editors can more easily edit a caption (#1580)
* SHS-5661: Remove legacy fields (#1577)
* SHS-4929: Hide caption/credits on images in default views (#1578)
* SHS-5675: Helping users avoid 'Table cell missing context (e.g. headings) A11y warning (#1579)
cienvaras added a commit that referenced this pull request Aug 22, 2024
* SHS-5692: Implementation: New Spotlight design for Colorful and Traditional (#1589)

* feat(shs-5692): add new field for variantion style and start implementing new styles

* feat(shs-5692): styles for colorful

* feat(shs-5692): finish styles for colorful

* feat(shs-5692): finish styles for colorful

* feat(shs-5692): finish styles for traditional

* feat(shs-5692): finish styles for traditional

* fix(shs-5692): fix on colorful

* fix(shs-5692): image dimensions and height when spotlits are in sliders

* fix(shs-5692): refactor spotlight style field class assignation

---------

Co-authored-by: Mari Nez <mariannuar@gmail.com>
Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>

* SHS-5772 - Regression: External Links on Postcards are wrong color (#1591)

* fix(shs-5772): regression in external links in postcards

* fix(shs-5772): update card title link styles

---------

Co-authored-by: Mari Nez <mariannuar@gmail.com>
Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>

* SHS-5629: Editors can more easily edit a caption (#1580)

* feat(SHS-5629): Add patch to allow editing of media items in a modal from a node edit page

* fix(SHS-5629): Remove todo in patch file

* SHS-5661: Remove legacy fields (#1577)

* feat(SHS-5661): Remove paragraph field Overlay Color on banner image with text box

* feat(SHS-5661): Remove other references to the overlay field

* feat(SHS-5661): Add update hook to remove legacy field

* fix(SHS-5661): Fix linting error

* fix(shs-5661): remove hero-text-overlay variants and update viewmode config

---------

Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>

* SHS-4929: Hide caption/credits on images in default views (#1578)

* feat(shs-4929): add new media view mode and update news default views

* feat(shs-4929):  update people default views

* feat(shs-4929):  update publications default views

---------

Co-authored-by: Mari Nez <mariannuar@gmail.com>
Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>

* refactor(SHS-5629): Switch local patch for remote patch, add patch to fix missing save button on media modal

* SHS-5675: Helping users avoid 'Table cell missing context (e.g. headings) A11y warning (#1579)

* feat(SHS-5675): Add default table header rows to new tables in CKEditor

* fix(SHS-5675): Fix linting error

* docs(SHS-5675): Add comments describing change and source of change.

* feat(SHS-5675): Update table heading color in CKEditor

* fix(shs-5675): change color variable for table headings

---------

Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>

* fix(shs-5629): revert composer.lock changes

* fix(SHS-5629): Possible fix for dialog box too low on screen

* fix(SHS-5629): Fix dialog too low on screen issue (clear browser cache to fix)

* fix(shs-5629): remove unnecesary change

* fix(shs-5629): update patch

---------

Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>
Co-authored-by: Mariana Núñez <48533432+mariannuar@users.noreply.github.com>
Co-authored-by: Mari Nez <mariannuar@gmail.com>

* feat(shs-5691): add new color pallete-for-traditional

* feat(shs-5691): add new color palette-for-traditional

* fix(shs-5691): update secondary and secondary active colors

---------

Co-authored-by: Mari Nez <mariannuar@gmail.com>
Co-authored-by: Andrés Díaz Soto <andres.diaz.soto@gmail.com>
Co-authored-by: Marc Berger <107938318+codechefmarc@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants