Skip to content

Conversation

@imantsk
Copy link
Member

@imantsk imantsk commented Nov 16, 2025

TL;DR

  • Improved vendor dependency management (PRO)
  • Enhanced UI for cloud snippets
  • Refined styling for back navigation

What changed?

  • Removed .browserslistrc file to prevent compatibility issues
  • Improved TinyMCE integration with custom scissors icon
  • Enhanced Imposter plugin integration to namespace-prefix vendor dependencies under Code_Snippets\Vendor\ to prevent conflicts with other plugins
  • Updated autoloader to dynamically remove original namespace PSR-4 mappings at runtime
  • Enhanced cloud snippets UI with improved card layout, better action buttons, and fixed pagination
  • Added back navigation button styling with arrow indicator

How to test?

  • Test the cloud snippets interface to verify the improved card layout and pagination
  • Check the back navigation from the snippet edit screen
  • Verify the TinyMCE integration with the new scissors icon
  • Test with a plugin that uses the same libraries to ensure no conflicts occur (PRO)
  • Check RTL support for proper display of UI elements

@imantsk imantsk added the run-tests Trigger automated tests label Nov 16, 2025

33% {
transform: rotate(calc(7deg * var(--cs-direction-multiplier)));
transform: rotate(7deg);
Copy link
Member

Choose a reason for hiding this comment

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

What's the purpose behind removing these changes? Were they causing issues?

Copy link
Member Author

Choose a reason for hiding this comment

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

No particolar reason. I just synced back the styles from what was in the pro repo. Also maybe a bit easier to understand

// Base64-encoded version of menu-icon.svg
const scissorsIcon =
'data:image/svg+xml;base64,' +
'PHN2ZyB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIGZpbGw9Im5vbmUiIHZpZX' +
Copy link
Member

Choose a reason for hiding this comment

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

I'm a little concerned this dump of base64 code might cause false-positives in security plugins (my concerns may be unfounded)

Copy link
Member Author

@imantsk imantsk Nov 17, 2025

Choose a reason for hiding this comment

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

I taught about that too.. i wanted to make a function that takes the actual file and encodes it in base64 ar required but somethig wasnt working right, so i ended up doing it like this.. maybe you could help with that?

@louiswol94 louiswol94 merged commit 7e92ad0 into core Nov 17, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-tests Trigger automated tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants