Skip to content
This repository was archived by the owner on Dec 11, 2017. It is now read-only.

Redesign warning/error messages#62

Merged
westonruter merged 1 commit intomasterfrom
add/error-messages
Aug 28, 2017
Merged

Redesign warning/error messages#62
westonruter merged 1 commit intomasterfrom
add/error-messages

Conversation

@obenland
Copy link
Copy Markdown
Member

Uses WP Admin color and icon scheme for warnings and errors.

Fixes #44, fixes #45.

After:
screen shot 2017-08-28 at 12 35 54 pm
Two messages stacked.

screen shot 2017-08-28 at 12 37 19 pm

Uses WP Admin color and icon scheme for warnings and errors.

Fixes #44, fixes #45.
@obenland obenland requested a review from melchoyce August 28, 2017 10:43
Copy link
Copy Markdown
Member

@westonruter westonruter left a comment

Choose a reason for hiding this comment

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

Looks so much better. 👍

@westonruter westonruter merged commit 18d46c4 into master Aug 28, 2017
@westonruter westonruter deleted the add/error-messages branch August 28, 2017 17:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Redesign error states Replace lint warning icon with dashicon

2 participants