search

vijaygill / wg-ui-plus

Wireguard VPN + Management Web-UI with fine-grained control over access.
MIT License
2 stars 0 forks source link

Applied icons to some elements. #21

Closed vijaygill closed 5 months ago

vijaygill commented 5 months ago

@thecoshman - nothing much. purely cosmetic.

thecoshman commented 5 months ago

Could not resolve "primeicons/primeicons.css"

Does something need to be committed?

vijaygill commented 5 months ago

Try npm insatall in the shell? This could mean we need to handle installing packages when the container starts (app or dev shell). But need more investigation.

thecoshman commented 5 months ago

That looks to have done the trick. I had to then restart the ng container. I think I know a way where npm install can automatically run when you start that container - does mean you'd have exit it after pulling then restart it, but I think that's tolerable

thecoshman commented 5 months ago

As it stands though... honestly not too sure if I'm a fan of these icons over the text that was in place. Maybe it's just the choice of icons, they seem a bit too hard and unclear... though maybe 'tool tips' would help. But this is more of a cosmetic thing at this point, so probably a branch that can be left for now

vijaygill commented 5 months ago

As it stands though... honestly not too sure if I'm a fan of these icons over the text that was in place. Maybe it's just the choice of icons, they seem a bit too hard and unclear... though maybe 'tool tips' would help. But this is more of a cosmetic thing at this point, so probably a branch that can be left for now

I just tried the icons provided by PrimeNG organiation itseelf (https://primeng.org/icons). Too limited.

I will try font-awesome some other time. I tried last night and reverted the change for some reason.