Update styles #15

Merged
merged 1 commit into from Feb 28, 2014

Conversation

Projects
None yet
3 participants
Contributor

nathanstaines commented Feb 27, 2014

Update the styles so the status messages are less intrusive during development.

@nathanstaines nathanstaines Update styles
Update styles some more
efa327e
Member

jhnns commented Feb 27, 2014

👍

@sokra sokra added a commit that referenced this pull request Feb 28, 2014

@sokra sokra Merge pull request #15 from nathanstaines/master
Update styles
09aa395

@sokra sokra merged commit 09aa395 into webpack:master Feb 28, 2014

Owner

sokra commented Feb 28, 2014

Thanks, looks much better 😄

Contributor

nathanstaines commented Feb 28, 2014

No problem, just glad I could help. Don't forget to push the new version to npm ;)

Owner

sokra commented Feb 28, 2014

I tried... npm doesn't like me. There was a weird error telling me there is a self signed cert in the cert chain... huh?

Owner

sokra commented Feb 28, 2014

ok is published

Contributor

nathanstaines commented Feb 28, 2014

Yeah I ran into that same issue today myself.

Also, I just realised I applied the wrong amount of padding to the header and error messages, it's only a small thing but one that's bound to irritate me every time I see it.

I could submit another pull request or if you'd prefer you could make the following adjustments in style.css... or simply leave it, it's up to you.

.header {
  ...
  padding: 0 10px;
  ...
}

#errors {
  ...
  padding: 10px;
  ...
}

@sokra sokra added a commit that referenced this pull request Feb 28, 2014

@sokra sokra fixes padding #15 304a25a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment