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

UI Localization #259

Open
wants to merge 21 commits into
base: v1.4
Choose a base branch
from

Conversation

sergikhizanishvili
Copy link
Contributor

No description provided.

galatanovidiu and others added 14 commits November 18, 2024 19:09
Add a check to return early from the `getSelectedGutenbergBlock` function if the `wp` object is undefined. This change guards against potential runtime errors when `wp` is not available in the execution context.
Added a check to return early if wp is undefined to avoid potential errors. This ensures the callback is only executed when the wp object is properly initialized.
Reorganized the React component hierarchy to place `ChatApp` inside a shadow DOM root element. Added an ID to the shadow DOM root element for improved identification and potential styling. Simplified the nesting structure for better readability and maintainability.
Updated the container for DropdownMenuPrimitive.Portal to target the shadow DOM element within the '#agentwp-chat-shadow-root' instead of the global window object. This change ensures the dropdown menu integrates properly within the encapsulated shadow DOM.
ENG-2065 - move dropdown inside the shadow root
@greghunt greghunt changed the base branch from main to v1.4 November 22, 2024 15:12
greghunt and others added 4 commits November 22, 2024 10:13
Update base CSS with base wp admin font size anc color
ENG-2050: Add a check for unexecuted actions to display the thinking component
@sergikhizanishvili sergikhizanishvili changed the title **WIP** UI Localization UI Localization Nov 26, 2024
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.

3 participants