concerto / concerto-weather

Concerto 2 Weather Plugin
Other
6 stars 3 forks source link

Huge owf #35

Open daniel-sousa-me opened 7 years ago

daniel-sousa-me commented 7 years ago

Why are you using owf-5x class?

This makes it huge.

mfrederickson commented 7 years ago

@garci66 I think this question was meant for you.

garci66 commented 7 years ago

I guess it can be easily replaced for a smaller font size. I though the weathericons font was also using a relatively large face. Maybe we should add a size selector? I was relying on concerto's autosize though so its a bit hard to know the exact size.

daniel-sousa-me commented 7 years ago

The owf-5x class makes them way bigger than the text. Then the autosize makes the text small.

Iirc, if you just remove the class, it becomes the appropriate size (the other icon set doesn't have that class and it looks good). Though adding an option should be easy.

garci66 commented 7 years ago

We can change the default class indeed. I'm not even sure why we went that way. Regardless, keep in mind you can also use the format string to wrap different parts of the text in CSS / styles, so we can control things separately.