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

SUP-192 | @rebeccahongsf | update media caption link styles #8

Merged
merged 6 commits into from
May 3, 2024

Conversation

rebeccahongsf
Copy link
Contributor

READY FOR REVIEW

Summary

  • update media caption link styles

Review By (Date)

  • When possible

Review Tasks

Setup tasks and/or behavior to test

  1. Check out this branch
  2. Rebuild Cache and import config drush cr ; drush ci
  3. Create a basic page with a media with caption paragraph type
  4. That the link is Stone Dark and on hover/focus, it is Archway Dark
  5. Review code 🪴

Site Configuration Sync

  • Is there a config:export in this PR that changes the config sync directory? Nope

Front End Validation

Associated Issues and/or People

  • SUP-192

@rebeccahongsf rebeccahongsf requested a review from pookmish May 2, 2024 23:14
},
colors: {
'press-indigo': '#003D69',
'press-grass': '#00593E',
Copy link
Contributor Author

@rebeccahongsf rebeccahongsf May 2, 2024

Choose a reason for hiding this comment

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

FYI: I stand corrected on this. I must've tested it incorrectly since the way @jenbreese had this configured originally before I made these changes in PR #5 was correct! 😃

Copy link
Contributor

Choose a reason for hiding this comment

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

How about adding "Magenta" and "Steel" while we're here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was able to locate the new colors press-sand and press-bay-light-light. Do you know where I can find the Magenta and Steel hex codes?
image

Copy link
Contributor

Choose a reason for hiding this comment

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

No idea. We'll come back to it

src/components/elements/link.tsx Outdated Show resolved Hide resolved
},
colors: {
'press-indigo': '#003D69',
'press-grass': '#00593E',
Copy link
Contributor

Choose a reason for hiding this comment

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

How about adding "Magenta" and "Steel" while we're here?

@rebeccahongsf rebeccahongsf requested a review from pookmish May 3, 2024 18:22
@rebeccahongsf
Copy link
Contributor Author

FYI: I'm looking into the lint error 👀

@rebeccahongsf rebeccahongsf merged commit e00d526 into 1.x May 3, 2024
1 check passed
@rebeccahongsf rebeccahongsf deleted the feature/SUP-192--mediaCaption branch May 3, 2024 19:37
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.

2 participants