Skip to content

[IMP] odoo_theme: add rotation classes#17967

Closed
fw-bot wants to merge 1 commit into
saas-19.2from
saas-19.2-19.0-phone-icon-rotate-nikbr-527364-fw
Closed

[IMP] odoo_theme: add rotation classes#17967
fw-bot wants to merge 1 commit into
saas-19.2from
saas-19.2-19.0-phone-icon-rotate-nikbr-527364-fw

Conversation

@fw-bot
Copy link
Copy Markdown
Collaborator

@fw-bot fw-bot commented May 18, 2026

For 19.0 and up there is a need to adjust the rotation of the font awesome icons (specifically fa-phone) to match the ui in the Odoo phone app.

Rather than creating a one off class for the phone icon, I used a sass loop to generate rotation classes in 5 degree increments so future icon adjustments can happen easily.

Also removed some old CSS that was for internet explorer 9 with rotation issues.

image

Forward-Port-Of: #17827

@robodoo
Copy link
Copy Markdown
Collaborator

robodoo commented May 18, 2026

Pull request status dashboard

@fw-bot
Copy link
Copy Markdown
Collaborator Author

fw-bot commented May 18, 2026

This PR targets saas-19.2 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

robodoo pushed a commit that referenced this pull request May 19, 2026
For 19.0 and up there is a need to adjust the rotation of the font awesome icons (specifically fa-phone) to match the ui in the Odoo phone app. 

Rather than creating a one off class for the phone icon, I used a sass loop to generate rotation classes in 5 degree increments so future icon adjustments can happen easily. 

Also removed some old CSS that was for internet explorer 9 with rotation issues. 

<img width="1705" height="1042" alt="image" src="https://github.com/user-attachments/assets/39b44197-177f-4874-92ad-c03a67b7e8f7" />

closes #17967

Forward-port-of: #17827
Signed-off-by: Antoine Vandevenne (anv) <anv@odoo.com>
Signed-off-by: Niki Brown (nikbr) <nikbr@odoo.com>
@robodoo robodoo closed this May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants