Skip to content

Conversation

@Eetwalt
Copy link
Collaborator

@Eetwalt Eetwalt commented Jan 27, 2026

Summary

This PR addresses and/or fixes:

  • preloading html2canvas when clicking Share for better performance
  • include alternative .grouptable selector for image exports
  • fix image rendering if user is zoomed in/out by resetting the zoom for the image
  • fix copy-to-clipboard failing due to the window losing focus mid copying
  • fix hide info icon from exported images

How did you test this change?

dev tools

Copilot AI review requested due to automatic review settings January 27, 2026 10:38
@Eetwalt Eetwalt added the javascript Changes to JavaScript files label Jan 27, 2026
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 fixes several image export bugs including rendering issues, clipboard functionality, and performance optimizations. The changes refactor the export flow to better handle clipboard operations and apply DOM fixes before canvas capture.

Changes:

  • Added preloading of html2canvas library when Share dropdown is opened for improved performance
  • Fixed image rendering issues by resetting zoom scale and applying SVG fixes for info icons
  • Refactored clipboard copy to use blob promises, preventing failures from window focus loss
  • Added .grouptable selector support and duplicate element detection during export

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

Copy link
Collaborator

@ElectricalBoy ElectricalBoy left a comment

Choose a reason for hiding this comment

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

seems reasonable

@Eetwalt Eetwalt merged commit a913c6a into main Jan 27, 2026
6 checks passed
@Eetwalt Eetwalt deleted the export-fix2 branch January 27, 2026 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Changes to JavaScript files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants