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 inline code styling when in a link #103

Merged
merged 4 commits into from
Mar 26, 2024
Merged

Conversation

MattIPv4
Copy link
Contributor

Type of Change

  • SCSS Styling: Code/links

What issue does this relate to?

N/A

What should this PR do?

Fix the styling of inline code that is within a link, so that the text in the inline code is visible while the link is being hovered. To allow for the code background and the link background to layer properly alongside the code text, I switched the background on inline code elements to being a pseudo-element, so I can layer it behind the background of the link.

Also, while I was here, I tweaked the alignment of the copy button in code blocks using Prism, as it looked rather off-center vertically in code blocks that had a single line of code.

What are the acceptance criteria?

Links on their own continue to look visually correct.

Inline code on its own continues to look visually correct.

Inline code with a variable inside continues to look visually correct.

Links with inline code inside continue to look visually correct.

Code blocks continue to look visually correct.

@MattIPv4 MattIPv4 added bug Something isn't working patch Change is SEMVER patch labels Mar 25, 2024
@MattIPv4 MattIPv4 force-pushed the MattIPv4/fix-code-in-link branch from 1f3049a to d8c25d1 Compare March 25, 2024 17:37
@MattIPv4 MattIPv4 merged commit fc90098 into master Mar 26, 2024
5 checks passed
@MattIPv4 MattIPv4 deleted the MattIPv4/fix-code-in-link branch March 26, 2024 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working patch Change is SEMVER patch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants