Skip to content

Comments

Remove Copy button from playground editor#16508

Merged
pelikhan merged 2 commits intomainfrom
copilot/remove-copy-button-editor
Feb 18, 2026
Merged

Remove Copy button from playground editor#16508
pelikhan merged 2 commits intomainfrom
copilot/remove-copy-button-editor

Conversation

Copy link
Contributor

Copilot AI commented Feb 18, 2026

Removes the Copy button and all associated code from the playground editor UI (docs/public/editor/index.html).

Changes

  • HTML: Removed Copy button element and feedback toast notification
  • CSS: Removed .copy-feedback styles (~16 lines)
  • JavaScript: Removed copyBtn variable, event handler, and state management (~30 lines)

Visual

Editor toolbar without Copy button

Header toolbar now shows: Auto-compile toggle → Compile button → Theme toggle

Users can still copy output via native browser selection (Ctrl+C).

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • telemetry.astro.build
    • Triggering command: /opt/hostedtoolcache/node/24.13.0/x64/bin/node node /home/REDACTED/work/gh-aw/gh-aw/docs/node_modules/.bin/astro dev (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Remove Copy button from editor in docs (index.html)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove copy button from editor in docs Remove Copy button from playground editor Feb 18, 2026
Copilot AI requested a review from pelikhan February 18, 2026 03:29
@pelikhan pelikhan marked this pull request as ready for review February 18, 2026 03:32
Copilot AI review requested due to automatic review settings February 18, 2026 03:32
@pelikhan pelikhan merged commit 67fe116 into main Feb 18, 2026
@pelikhan pelikhan deleted the copilot/remove-copy-button-editor branch February 18, 2026 03:32
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the Copy button functionality from the playground editor UI to simplify the interface. Users can still copy output using native browser selection (Ctrl+C). The changes are focused on cleaning up UI elements and associated code from a single file.

Changes:

  • Removed Copy button HTML element and associated feedback toast notification
  • Removed CSS styles for the copy feedback toast (~16 lines)
  • Removed JavaScript for Copy button functionality including event handler and clipboard operations (~30 lines)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -558,7 +530,6 @@
outputPlaceholder.style.display = 'flex';
outputPlaceholder.textContent = 'Compiled YAML will appear here';
currentYaml = '';
Copy link

Copilot AI Feb 18, 2026

Choose a reason for hiding this comment

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

The currentYaml variable is no longer used after removing the Copy button functionality. This variable was only read by the removed copy event handler. Consider removing the currentYaml variable declaration (line 378) and all its assignments (lines 532 and 553) to avoid maintaining unused state.

Copilot uses AI. Check for mistakes.
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