feature/code-block-language-header #1389
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
sfc-gh-dmatthews
force-pushed
the
12-14-feature_code-block-language-header
branch
from
687d1c2 to
f50584c
Compare
sfc-gh-dmatthews
requested review from
sfc-gh-kmcgrady and
sfc-gh-tteixeira
as code owners
| } | ||
|
|
||
| .Container section :global(.code-block-header) { | ||
| @apply hidden !important; |
There was a problem hiding this comment.
question: is there a way to do this without the !important?
There was a problem hiding this comment.
I struggled to get this apply. I tried hiding it at different levels, using different selectors, but had to end up with !important, but I could have missed something.
|
|
||
| const Header = ( | ||
| <div className={classNames(styles.Header, "code-block-header")}> | ||
| {showLanguage && ( |
There was a problem hiding this comment.
question: If showLanguage is false, then it looks like Header will just be a blank div with a height of 2.5rem. Is that expected?
There was a problem hiding this comment.
The copy button should still be visible. (After the feature is merged, this should be a rare case because I will update the docs. IIRC, I will only use "None" for directory structures or text files, and I will add a "Directory structure" description or "requirements.txt" in those instances, which is the next PR.)
For example, this is a "None" example that will have a description enabled in the next PR.
sfc-gh-dmatthews
deleted the
12-14-feature_code-block-language-header
branch
📚 Context
🧠 Description of Changes
Current:
Insert screenshot of existing UI/code here
Revised:
Insert screenshot of your updated UI/code here
💥 Impact
Size:
🌐 References
Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.