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

Fixed #1276 -- Added copy buttons to console and shell commands. #1447

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nvishnya
Copy link

@nvishnya nvishnya commented Dec 6, 2023

Fixes #1276

Although there is a PR pending, added one as well, hope it's okay.

Added copy buttons for console and shell commands. Also added copy buttons on the Download page.
Minimum code changes, no new styles added, button adding functionality is moved to a separate function, so it can be further reused with different code blocks.

test

Comments aren't copied (like in an example below), nor are the promts.
12023-12-06_21-30

Copy link
Sponsor Member

@sabderemane sabderemane left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution! I believe it's better to start your contribution from @prashantpandey9's PR in order to have the initial contribution and the co-author.

@@ -3624,6 +3624,11 @@ ul.corporate-members li {
}

.code-block-caption,
.highlight-console,
.c-content-win,
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

I think it's missing .c-content-unix

Copy link
Author

Choose a reason for hiding this comment

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

No need as there is a child div with .highlight-console for every section with .c-content-unix.
2024-06-10_13-52

if (hasClass('code-block-caption') ||
hasClass('snippet') ||
hasClass('highlight-console') ||
hasClass('c-content-win') ||
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

Same, I think it's missing .c-content-unix

@nvishnya
Copy link
Author

I saw @prashantpandey9's PR, but decided to start the contribution from scratch. I think it's better not to modify existing styles but to reuse them where necessary.

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.

Add Click to copy text for commands present in the official Django website.
2 participants