Skip to content

Master mysterious egg loco 1 #4489

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

Draft
wants to merge 3 commits into
base: master-mysterious-egg
Choose a base branch
from

Conversation

loco-odoo
Copy link

No description provided.

@robodoo
Copy link

robodoo commented Apr 25, 2025

This PR targets the un-managed branch odoo-dev/odoo:master-mysterious-egg, it needs to be retargeted before it can be merged.

@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch 2 times, most recently from 5906a60 to 1540589 Compare April 28, 2025 08:44
this.dialogService = this.services.dialog;
const editableEls = findOEditable(this.editable);
this.elToTranslationInfoMap = new Map();
const translatedAttrs = ["placeholder", "title", "alt", "value"];
Copy link
Author

Choose a reason for hiding this comment

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

Note: for the moment, there is a problem when a form field has an input attribute. I will solve it in another PR; the problem is that interactions that should not be started in edit mode are somehow started and destroyed when entering translate mode.

@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch from 1540589 to 4674893 Compare April 29, 2025 07:24
@@ -0,0 +1,72 @@
import { Dialog } from "@web/core/dialog/dialog";
Copy link
Author

Choose a reason for hiding this comment

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

Just copy pasted from website

@loco-odoo
Copy link
Author

loco-odoo commented Apr 29, 2025

To fix in stable:

  • Add a title to an image
  • Translate the title with a special character (example: "<")
  • Click on "Ok"
  • Try to retranslate the attribute => it is encoded as &lt;

Task: https://www.odoo.com/odoo/project/49/tasks/4759395

@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch from 4674893 to bdc5c86 Compare April 29, 2025 08:43
@@ -37,9 +49,6 @@ export class SetupEditorPlugin extends Plugin {
el.setAttribute("data-editor-message", _t("DRAG BUILDING BLOCKS HERE"));
}
});

// Set the `contenteditable` attribute on the editables.
this.setContenteditable();
Copy link
Author

@loco-odoo loco-odoo Apr 29, 2025

Choose a reason for hiding this comment

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

Done at the normalize so useless at the end of the setup

@ged-odoo ged-odoo force-pushed the master-mysterious-egg branch from e24556e to 897f4ab Compare April 29, 2025 18:30
@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch from bdc5c86 to 7f8e782 Compare April 30, 2025 07:25
- translate attributes
- translate selections
- add a message when trying to translate not editable elements
- add a message to explain the translation colors
@loco-odoo loco-odoo force-pushed the master-mysterious-egg-loco-1 branch from 7f8e782 to 27f8f1b Compare April 30, 2025 07:28
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