Monday, March 15, 2021
2 changes · master
Resolved issues and error corrections
This fix removes survey report styling rules that were accidentally applied across all Odoo reports. It helps prevent unrelated reports from having broken layouts and reduces the risk of future style conflicts.
Original PR description
This commit removes 2 css rules that were applied on ALL odoo reports by mistake. Indeed, adding rules on "*" and "body" will most likely break other reporting layouts and is very dangerous / unintended. If these rules are needed for the survey reports, then it should be fixed to make them only applied to those reports instead. We also took this opportunity to get rid of some scss variables. These variables names were too "global" and could also conflict with other reporting css rulesets. If we want to use variables for that report, they should be correctly pre-fixed to avoid collisions. Source: 212b107fa155778e2b2063b66bb88517aba80498 Task-2341847 Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: -- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr
Selecting icons in the website editor's media dialog now behaves correctly and displays the selected state properly. Icons are also centered, making the media picker clearer and easier to use.
Original PR description
The code added in bb65b742e7eec000248b7df0f048dda61ac3ff5a to update style for multi image selection is not compatible with .fa icons and causes a strange behaviour when trying to select one. The goal of this PR is to fix style for selected icons on Media Dialog. task-2326573