docs: Updated index page to make it look more attractive #4855

Merged
merged 1 commit into from Oct 23, 2016

Projects

None yet

6 participants

@laf
Member
laf commented Oct 22, 2016

Please note

Please read this information carefully. You can run ./scripts/pre-commit.php to check your code before submitting.

Looks like:

image

@laf laf docs: Updated index page to make it look more attractive
fc316dc
@laf laf added the Documentation label Oct 22, 2016
@laf
Member
laf commented Oct 22, 2016

Yes it's html based rather than markdown but we shouldn't need to change this very often. It's just a nicer way to intro people to some quick win documents without a wall of text.

@scrutinizer-notifier

The inspection completed: No new issues

@Rosiak
Rosiak approved these changes Oct 22, 2016 View changes
@murrant
Contributor
murrant commented Oct 23, 2016

Looks great. Just a nitpick, the shade of red looks a little off since there is already red on the page. Don't make it the same shade of red though 😀.

@laf
Member
laf commented Oct 23, 2016

It's just using bootstrap color, the red in the menu is a little too bright though so wasn't sure it was the best choice.

@murrant
Contributor
murrant commented Oct 23, 2016

Looks good for now, can tweak later if anyone has a better idea.

@murrant murrant merged commit 1d2bc5a into librenms:master Oct 23, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@laf laf deleted the laf:docs-index branch Oct 23, 2016
@huegelc
huegelc commented Oct 24, 2016

Looks good indeed. As you mentioned the red in the navbar:

librenms_docs

looks like if you align both colors.

@laf
Member
laf commented Oct 24, 2016

The red from the nav is taken from the webui and main website which in turn are taken from the logo as an fyi

@huegelc
huegelc commented Oct 24, 2016

OK, but this doesn´t look bad either;):

librenms_docs

.text-danger {
    color: #da223a;
}
@laf
Member
laf commented Oct 24, 2016

Just find it a bit bright that's all

@murrant
Contributor
murrant commented Oct 25, 2016

I agree, to some extent, but like I said, I'm open to a better color :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment