Permalink
Browse files

fix(icons): sizes of Font awesome icons are now more consistent with …

…old icons

 - Individually increases the size of some icons that were looking too small
 - Removes fixed width and height from icon containers
 - Increases default icon font size from 16px to 18px

Fixes #8733, #8861
  • Loading branch information...
juho-jaakkola committed Aug 14, 2015
1 parent eb5837b commit 11386003f9793fda1ce11c1ef59de9027dac99ee
Showing with 20 additions and 7 deletions.
  1. +13 −0 views/default/elements/fa.css
  2. +5 −7 views/default/elements/icons.css.php
  3. +2 −0 views/default/elgg.css.php
@@ -0,0 +1,13 @@
/**
* Hack that reduces size inconsistencies within Font awesome icons
*/
.fa-mobile-phone:before,
.fa-mobile:before,
.fa-remove:before,
.fa-close:before,
.fa-times:before,
.fa-caret-down:before,
.fa-caret-down:before {
font-size: larger !important;
}
@@ -14,9 +14,7 @@
*************************************** */
.elgg-icon {
color: #CCC;
font-size: 16px;
width: 16px;
height: 16px;
font-size: 18px;
margin: 0 2px;
}
@@ -67,20 +65,20 @@
.elgg-avatar-tiny > a > img {
width: 25px;
height: 25px;
/* remove the border-radius if you don't want rounded avatars in supported browsers */
border-radius: 3px;
background-clip: border;
background-size: 25px;
}
.elgg-avatar-small > a > img {
width: 40px;
height: 40px;
/* remove the border-radius if you don't want rounded avatars in supported browsers */
border-radius: 5px;
background-clip: border;
background-size: 40px;
}
@@ -39,6 +39,7 @@
* layout_objects - lists, content blocks, notifications, avatars
* layout - page layout
* misc - to be removed/redone
* fa - hacks to reduce size inconsistencies in Font awesome icons
*******************************************************************************/
echo elgg_view('elements/typography.css', $vars);
@@ -51,6 +52,7 @@
echo elgg_view('elements/layout.css', $vars);
echo elgg_view('elements/misc.css', $vars);
echo elgg_view('elements/misc/spinner.css', $vars);
echo elgg_view('elements/fa.css', $vars);
// included last to have higher priority

0 comments on commit 1138600

Please sign in to comment.