Skip to content

🎨 Palette: Add section copy links to documentation#20

Open
MilakyS wants to merge 4 commits into
mainfrom
palette-docs-copy-link-17373943267461537362
Open

🎨 Palette: Add section copy links to documentation#20
MilakyS wants to merge 4 commits into
mainfrom
palette-docs-copy-link-17373943267461537362

Conversation

@MilakyS
Copy link
Copy Markdown
Member

@MilakyS MilakyS commented May 16, 2026

💡 What: Added "Copy Link" buttons to all documentation section headers in docs.html.
🎯 Why: Improves navigation and shareability by allowing users to quickly grab a direct link to a specific section or tabbed example.
Accessibility:

  • Uses semantic <button> elements.
  • Provides descriptive aria-label for screen readers.
  • Uses aria-hidden on icons to prevent redundant announcements.
    UX Delight:
  • Subtle link icon appears on section hover to avoid visual clutter.
  • Immediate visual feedback (icon change to checkmark) on successful copy.
  • Smooth transitions for hover states.

PR created automatically by Jules for task 17373943267461537362 started by @MilakyS

- Injected copy link buttons into dynamic documentation headers
- Implemented clipboard logic with 1.5s visual feedback (checkmark)
- Added hover-triggered CSS for copy buttons
- Modernized related JS syntax and refined CSS selectors
- Updated Palette journal with UX learnings

Co-authored-by: MilakyS <155958485+MilakyS@users.noreply.github.com>
@google-labs-jules
Copy link
Copy Markdown
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 16, 2026

Deploy Preview for melodious-queijadas-3cc817 ready!

Name Link
🔨 Latest commit 7ff0ff0
🔍 Latest deploy log https://app.netlify.com/projects/melodious-queijadas-3cc817/deploys/6a080eec57edd90008b8298a
😎 Deploy Preview https://deploy-preview-20--melodious-queijadas-3cc817.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 16, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
mks-lang 7ff0ff0 May 16 2026, 06:30 AM

google-labs-jules Bot and others added 3 commits May 16, 2026 06:02
- Added `typeof window === 'undefined'` guards to all IIFEs in `docs.js` and `docs-page.js`
- Replaced `async/await` with Promises in `docs.js` for better build-time compatibility
- Replaced `replaceAll` with global regex in `docs-page.js`
- Added leading semicolons to IIFEs to prevent concatenation errors

Co-authored-by: MilakyS <155958485+MilakyS@users.noreply.github.com>
- Replaced all `replaceAll` with global regex replacements across all JS files
- Added `typeof window === 'undefined'` guards to all IIFEs
- Added leading semicolons to IIFEs to prevent build-time concatenation errors
- Ensured `docs.js` uses Promises instead of `async/await` for better compatibility
- Wrapped `playground.js` in a defensive IIFE

Co-authored-by: MilakyS <155958485+MilakyS@users.noreply.github.com>
- Purged all `async/await` syntax in favor of Promise chains
- Replaced all `replaceAll` calls with global regular expressions
- Ensured all IIFEs have leading semicolons and defensive `window` guards
- Verified Docs Copy Link functionality remains intact
- Optimized asynchronous data loading in common.js, docs-page.js, changelog.js, and roadmap.js

Co-authored-by: MilakyS <155958485+MilakyS@users.noreply.github.com>
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.

1 participant