Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improved client overlay style #360

Merged
merged 1 commit into from
May 15, 2019
Merged

improved client overlay style #360

merged 1 commit into from
May 15, 2019

Conversation

jimblue
Copy link
Contributor

@jimblue jimblue commented May 14, 2019

This PR contains a fix for #359.

Motivation

The style of the client overlay is slightly change for better readability.

Breaking Changes

This PR doesn't introduce breaking changes

Additional Info

Here are some before after screenshot of the client overlay.

WARNING
before
before1
after
after1
ERROR
before
before2
after
after2

@jsf-clabot
Copy link

jsf-clabot commented May 14, 2019

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented May 14, 2019

Codecov Report

Merging #360 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #360   +/-   ##
=======================================
  Coverage   88.63%   88.63%           
=======================================
  Files           3        3           
  Lines         264      264           
  Branches       76       76           
=======================================
  Hits          234      234           
  Misses         30       30

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fe33d59...e2b49ff. Read the comment docs.

Copy link
Collaborator

@glenjamin glenjamin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I notice you changed some of the other colours too, would you mind doing a before/after on those too?

@jimblue
Copy link
Contributor Author

jimblue commented May 15, 2019

I notice you changed some of the other colours too, would you mind doing a before/after on those too?

You can see it for yellow and red on the above before after. (look at the background color of the tags)
For the others colors I keep the same direction: "better readability with pop/constrasty/stylish colors".
It's a discret change put enough to give better readability.
Let me know if you need more...

@glenjamin
Copy link
Collaborator

I’d like to leave screenshots of all the changed colours in the issue as a historical record

Did you have some sample output you used to select the changes to the ANSI colours?

@jimblue
Copy link
Contributor Author

jimblue commented May 15, 2019

Screenshots are gonna be difficult as only two colors are used in the client overlay. (red/yellow and you already have thoses screenshot)

But, what I can do on photoshop is a color palette of before after, what do you think?

@glenjamin
Copy link
Collaborator

How were the new ANSI colours selected? I had assumed you had some sample output which used a variety of colours and demonstrated the need for improvement

@jimblue
Copy link
Contributor Author

jimblue commented May 15, 2019

Here is a visual way to show the change 😄

Untitled

@glenjamin
Copy link
Collaborator

Perfect, thanks!

@glenjamin glenjamin merged commit 82dd483 into webpack-contrib:master May 15, 2019
@glenjamin
Copy link
Collaborator

Published as v2.25.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants