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] web: highlight clickable pivot cells on hover #29632

Open
wants to merge 1 commit into
base: saas-12.1
from

Conversation

Projects
None yet
4 participants

@robodoo robodoo added the seen 🙂 label Dec 19, 2018

@C3POdoo C3POdoo added the RD label Dec 19, 2018

@robodoo robodoo added the CI 🤖 label Dec 19, 2018

@ans-odoo ans-odoo force-pushed the odoo-dev:saas-12.1-highlight-pivot-cell-ans branch Dec 20, 2018

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Dec 20, 2018

@ans-odoo ans-odoo force-pushed the odoo-dev:saas-12.1-highlight-pivot-cell-ans branch Dec 20, 2018

@robodoo robodoo removed the CI 🤖 label Dec 20, 2018

@ans-odoo ans-odoo force-pushed the odoo-dev:saas-12.1-highlight-pivot-cell-ans branch Dec 20, 2018

@robodoo robodoo added the CI 🤖 label Dec 20, 2018

@ans-odoo ans-odoo force-pushed the odoo-dev:saas-12.1-highlight-pivot-cell-ans branch Dec 20, 2018

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Dec 20, 2018

@ans-odoo ans-odoo force-pushed the odoo-dev:saas-12.1-highlight-pivot-cell-ans branch Dec 21, 2018

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Dec 21, 2018

@dja-odoo

This comment has been minimized.

Copy link
Contributor

commented Jan 8, 2019

LGTM

@Polymorphe57 Polymorphe57 force-pushed the odoo-dev:saas-12.1-highlight-pivot-cell-ans branch to 9736401 Feb 12, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Feb 12, 2019

@dja-odoo dja-odoo force-pushed the odoo-dev:saas-12.1-highlight-pivot-cell-ans branch from 9736401 to 52f802f Mar 6, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Mar 6, 2019

[FIX] web: highlight clickable pivot cells on hover
After a recent reorganization of the web client's components tree, the
class 'o_enable_linking' was added on the wrong element. This made the
css rule responsible of the highlighting of the clickable pivot cells
impossible to satisfy.

This commit fixes the issue by adding the class directly in the template
'PivotView'.

Additionally, this commit also makes visually clear when a field
selection menu-items is disabled (because it is already used) using
a 'not-allowed' cursor (which was removed in bs4). It is now again
possible to open a list view by clicking on a cell with zero value
(like in Odoo 11).

task-1916889

@dja-odoo dja-odoo force-pushed the odoo-dev:saas-12.1-highlight-pivot-cell-ans branch from 52f802f to e913618 Apr 24, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Apr 24, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.