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

[FIX] snippet editor for complex options (website_animate) #31440

Closed
wants to merge 2 commits into from

Conversation

qsm-odoo
Copy link
Contributor

opw-1929133

@qsm-odoo qsm-odoo self-assigned this Feb 26, 2019
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Feb 26, 2019
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Feb 26, 2019
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Feb 27, 2019
@qsm-odoo
Copy link
Contributor Author

@robodoo r+ rebase-ff

@robodoo
Copy link
Contributor

robodoo commented Feb 27, 2019

Merge method set to rebase and fast-forward

@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses merging 👷 and removed merging 👷 CI 🤖 Robodoo has seen passing statuses labels Feb 27, 2019
@robodoo
Copy link
Contributor

robodoo commented Feb 27, 2019

Staging failed: ci/runbot on f6bbf03266d94b047ce6bd629f8df54b2cc5b36b (view more at http://runbot.odoo.com/runbot/build/471324)

Commit odoo@c2b8d2a
made the mistake of considering each dropdown-menu individually while
all are considered to be the same when enabling a class. From saas-11.x,
commit odoo@8ca9233
will group by dropdown menu but that will be another fix to make.

Part of odoo#31440
opw-1929133
@qsm-odoo qsm-odoo force-pushed the 11.0-website-animate-fix-qsm branch from d76ebe8 to 2baf110 Compare March 1, 2019 14:17
@qsm-odoo
Copy link
Contributor Author

qsm-odoo commented Mar 1, 2019

@robodoo retry

@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses r+ 👌 and removed error 🙅 labels Mar 1, 2019
robodoo pushed a commit that referenced this pull request Mar 1, 2019
Commit c2b8d2a
made the mistake of considering each dropdown-menu individually while
all are considered to be the same when enabling a class. From saas-11.x,
commit 8ca9233
will group by dropdown menu but that will be another fix to make.

Part of #31440
opw-1929133
robodoo pushed a commit that referenced this pull request Mar 1, 2019
@robodoo
Copy link
Contributor

robodoo commented Mar 1, 2019

Merged, thanks!

@robodoo robodoo closed this Mar 1, 2019
@qsm-odoo qsm-odoo deleted the 11.0-website-animate-fix-qsm branch March 4, 2019 12:24
qsm-odoo added a commit to odoo-dev/odoo that referenced this pull request Mar 15, 2019
Commit odoo@8ca9233
did not think about submenus inside submenus, which is the case in the
website animate option. This made snippets potentially have multiple
animations on them, with undesired behaviors.

This commit also accordingly reviews the _setActive method for the
select-class option and also adapts following the recent forward-port of
odoo#31440

Related to task-1929133
robodoo pushed a commit that referenced this pull request Mar 15, 2019
Commit 8ca9233
did not think about submenus inside submenus, which is the case in the
website animate option. This made snippets potentially have multiple
animations on them, with undesired behaviors.

This commit also accordingly reviews the _setActive method for the
select-class option and also adapts following the recent forward-port of
#31440

Related to task-1929133

closes #31477

Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
Zynton pushed a commit to odoo-dev/odoo that referenced this pull request Aug 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants