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

Master phoenix rcdl 2 #3011

Merged
merged 2 commits into from Mar 27, 2024
Merged

Master phoenix rcdl 2 #3011

merged 2 commits into from Mar 27, 2024

Conversation

cammarosano
Copy link

No description provided.

@robodoo
Copy link

robodoo commented Mar 26, 2024

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

// Helpers for normalize
// --------------------------------------------------------------------------

preserveSelection() {

Choose a reason for hiding this comment

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

looks like this should be in selection plugin?

Copy link
Author

Choose a reason for hiding this comment

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

Probably.
But this is rather incomplete and only handles the use cases of list normalization so far...

Choose a reason for hiding this comment

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

it's fine to have incomplete implementation imo. and it's better to put code at the proper location, so there is a higher chance that someone finds it (and improve it if necessary) later

@@ -105,3 +79,19 @@ export function applyToTree(root, func) {
}
return modifiedRoot;
}

// For now, only a few cases are handled.
export function preservePos(node, offset) {
Copy link

Choose a reason for hiding this comment

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

move into a method of the list plugin

@ged-odoo ged-odoo merged commit 6a96e80 into master-phoenix Mar 27, 2024
@ged-odoo ged-odoo deleted the master-phoenix-rcdl-2 branch March 27, 2024 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants