aurelia-v-grid / vGrid

Aurelia-v-grid - npm source
MIT License
49 stars 10 forks source link

icons need more work #64

Closed vegarringdal closed 7 years ago

vegarringdal commented 7 years ago

Icons need more work

Maybe use "user ref" and have 1 svg users need to load, that way replacing icons will be easier

Anyways, design of my icons will need more work too, first time Ive ever made svg icons. So some look very bad atm

vegarringdal commented 7 years ago

@Thanood

this is how the icons look atm, will use https://jakearchibald.github.io/svgomg/ to get size of path data down, and also make them little bit smaler

image

image

image

image

vegarringdal commented 7 years ago

Been thinking about skipping this icons craziness on the menu, just have the sort icon, and + and minus for grouping, and X (but nicer for removing grouping

What to you think @Thanood ?

Thanood commented 7 years ago

I think there should also be more margin between icons and text. :smiley:

I agree that maybe the operators don't need an icon. Maybe it looks less "bloated" but it may also look good when you make them a bit smaller, as you already said. I'd try both and see how it looks.