fix: remove redundant font-family from .response-text .inline-code#339
fix: remove redundant font-family from .response-text .inline-code#339abhiramvsmg wants to merge 1 commit into
Conversation
|
|
|
Thanks for the cleanup! The reasoning is sound and I verified it end-to-end:
One thing to fix before merge: The declaration wasn't deleted cleanly — it was replaced with a line of trailing whitespace (and the indentation is off), leaving a dangling blank line inside the block: .response-text .inline-code {
background: var(--inline-code-bg);
padding: 0.15em 0.35em;
border-radius: 4px;
font-size: 0.88em;
/* <- leftover blank line with trailing spaces */
}Please remove the whole line instead so the block ends cleanly at |
|
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. |
Removed the redundant
font-family: var(--mono);declaration from.response-text .inline-codein style.css, since it's rendered as a<code>element and now inherits the same font from the new globalcode, prerule added in PR #328.Left
.file-pathand.tool-nameuntouched — they render as<span>and<strong>, not<code>/<pre>, so the global rule doesn't apply to them and removing theirfont-familywould break monospace rendering.Fixes #336