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

Add 15.0, FontAwesome 4.7 and fix typos #5

Merged
merged 1 commit into from
Apr 7, 2022
Merged

Add 15.0, FontAwesome 4.7 and fix typos #5

merged 1 commit into from
Apr 7, 2022

Conversation

fbarbe00
Copy link
Contributor

Purpose

As of version 15.0, Odoo is still using Font Awesome 4.7 for its icons.
It can hence be useful to have it as an option to have a coherent icon
with the rest of Odoo.

This commit adds support for FontAwesome 4.7, adds 15.0 to the list
(icon style has not changed since 12.0), and fixes a few typos.

Purpose
===========
As of version 15.0, Odoo is still using Font Awesome 4.7 for its icons.
It can hence be useful to have it as an option to have a coherent icon
with the rest of Odoo.

This commit adds support for FontAwesome 4.7, adds 15.0 to the list
(icon style has not changed since 12.0), and fixes a few typos.
@fbarbe00
Copy link
Contributor Author

@spilymp are there any plans to merge this? :)

@altela
Copy link

altela commented Apr 7, 2022

hi @spilymp
any ideas?

@spilymp spilymp merged commit cb0a1d1 into spilymp:main Apr 7, 2022
@spilymp
Copy link
Owner

spilymp commented Apr 7, 2022

Sincere thanks for your contribution and patience! I have merged your pull request. However, I did not understand the meaning of your additional JavaScript function. Therefore I have changed some things afterwards. If there are any errors or if this function serves an important purpose, please comment here or create a separate issue for it.

@fbarbe00
Copy link
Contributor Author

fbarbe00 commented Apr 8, 2022

Hi @spilymp , thanks for the merge! Everything seems to work correctly (except using fa fa-book in FontAwesome 5 instead of fas fa-book. Previously both worked I believe).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants