Added borders between elements on availability map. #1713

Merged
merged 1 commit into from Aug 18, 2015

Projects

None yet

2 participants

@SaaldjorMike
Member

Before: https://i.imgur.com/Ms4SUD6.png
After: https://i.imgur.com/1yyuH0B.png

Personally I'd like to have a lille border to separate the different boxes. If you guys don't like it, just let me know and I'll close the PR again. :-)

@f0o f0o self-assigned this Aug 18, 2015
@f0o f0o added the WebUI label Aug 18, 2015
@f0o
Member
f0o commented Aug 18, 2015

@SaaldjorMike isnt border:1px better?

@SaaldjorMike
Member

@f0o: Seems like using border:1px gives the following: https://i.imgur.com/gX4TkIJ.png

@f0o
Member
f0o commented Aug 18, 2015

@SaaldjorMike is the default then 'em' or what unit? I dislike having the border parameter without a unit. Could lead to different outcomes depending on the browser's assumptions of units..

@f0o f0o merged commit 443d19b into librenms:master Aug 18, 2015

1 check was pending

Scrutinizer Running Analyses
Details
@SaaldjorMike SaaldjorMike deleted the SaaldjorMike:map-border branch Jan 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment