Skip to content

icons not defined in less files? #4658

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

Closed
BerndWessels opened this issue Nov 8, 2015 · 7 comments
Closed

icons not defined in less files? #4658

BerndWessels opened this issue Nov 8, 2015 · 7 comments
Labels

Comments

@BerndWessels
Copy link

Hello
It seems that the icon definitions are not part of the ui-grid less files. Why is that? In my projects I only use less files and wanted to @import '~angular-ui-grid/less/main.less';. But to get that working I have to manually extract the icons from the ui-grid.css and add them to my own less files:

.ui-grid-tree-header-row {
font-weight: bold !important;
}

.ui-grid-icon-plus-squared:before {
content: '\c350';
}
/* '썐' */
.ui-grid-icon-minus-squared:before {
content: '\c351';
}
/* '썑' */
.ui-grid-icon-search:before {
content: '\c352';
}
/* '썒' */
.ui-grid-icon-cancel:before {
content: '\c353';
}
/* '썓' */
.ui-grid-icon-info-circled:before {
content: '\c354';
}
/* '썔' */
.ui-grid-icon-lock:before {
content: '\c355';
}
/* '썕' */
.ui-grid-icon-lock-open:before {
content: '\c356';
}
/* '썖' */
.ui-grid-icon-pencil:before {
content: '\c357';
}
/* '썗' */
.ui-grid-icon-down-dir:before {
content: '\c358';
}
/* '썘' */
.ui-grid-icon-up-dir:before {
content: '\c359';
}
/* '썙' */
.ui-grid-icon-left-dir:before {
content: '\c35a';
}
/* '썚' */
.ui-grid-icon-right-dir:before {
content: '\c35b';
}
/* '썛' */
.ui-grid-icon-left-open:before {
content: '\c35c';
}
/* '썜' */
.ui-grid-icon-right-open:before {
content: '\c35d';
}
/* '썝' */
.ui-grid-icon-angle-down:before {
content: '\c35e';
}
/* '썞' */
.ui-grid-icon-filter:before {
content: '\c35f';
}
/* '썟' */
.ui-grid-icon-sort-alt-up:before {
content: '\c360';
}
/* '썠' */
.ui-grid-icon-sort-alt-down:before {
content: '\c361';
}
/* '썡' */
.ui-grid-icon-ok:before {
content: '\c362';
}
/* '썢' */
.ui-grid-icon-menu:before {
content: '\c363';
}
/* '썣' */
.ui-grid-icon-indent-left:before {
content: '\e800';
}
/* '' */
.ui-grid-icon-indent-right:before {
content: '\e801';
}
/* '' */
.ui-grid-icon-spin5:before {
content: '\ea61';
}

Is this a bug or do I miss something here?

Cheers
Bernd

@crazyjat
Copy link

This does seem to be a bug and a big deal for those of us trying to use the less files.

Why has this not been addressed in almost a year?

Where do these styles come from?

@BerndWessels
Copy link
Author

I think this project is dead. Otherwise an issue like this would have caused a huge outcry.

@crazyjat
Copy link

crazyjat commented Jul 11, 2016

I'm confused. Are you claiming that the ui-grid project is dead? It is still actively developed and supported.

@BerndWessels
Copy link
Author

BerndWessels commented Jul 12, 2016

@crazyjat Since this issue has not been fixed in a year as you pointed out, I believe nobody can use this project properly using less. Without less there is no proper white-labeling. And as a result I doubt the project could seriously be considered for production. And a project that is not production ready for such a long time is dead to me at least.

@c0bra
Copy link
Contributor

c0bra commented Jul 12, 2016

The icons are defined by less that is generated by Fontello: http://fontello.com/

This was done when the icons were rapidly changing and having new ones added. Since they've stabilized quite a bit, the less could probably be used statically rather than generating it as part of the build.

@BerndWessels
Copy link
Author

@c0bra Thanks for the info. I moved almost all my projects to react and react-native now. But for everybody else still on angular I would suggest you provide proper less support for this project.
Cheers
Bernd

DeusEx-Machina pushed a commit to DeusEx-Machina/ui-grid that referenced this issue Oct 20, 2016
Added icon definitions into icons.less and removed fontello from build process

angular-ui#4658
dlgski pushed a commit that referenced this issue Nov 8, 2016
* Fixes #3832

* chore(less): Propose fix for #4658

Added icon definitions into icons.less and removed fontello from build process

#4658
@mportuga
Copy link
Member

mportuga commented Nov 8, 2016

Closed by #5762

@mportuga mportuga closed this as completed Nov 8, 2016
goblinbr pushed a commit to kugel-soft/ui-grid that referenced this issue May 24, 2018
* Fixes angular-ui#3832

* chore(less): Propose fix for angular-ui#4658

Added icon definitions into icons.less and removed fontello from build process

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

No branches or pull requests

5 participants