Closed fbarbe00 closed 2 years ago
@spilymp are there any plans to merge this? :)
hi @spilymp any ideas?
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.
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).
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.