-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add warning category #129
Add warning category #129
Conversation
eec0d86
to
bbdb8b6
Compare
@@ -178,7 +178,10 @@ | |||
<i class="fas fa-fw fa-exclamation-triangle" aria-hidden="true"></i> | |||
</div> | |||
<div class="pf-c-alert__title"> | |||
{{- warning | set_css_for_list -}} | |||
Warning category: {{- warning.category | capitalize -}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks ugly. Keep only Warning
as a title. Put the category value to a box or under the heading and don't use the "category" word at all.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That sounds great!
bbdb8b6
to
bb94541
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have generated the report and I have seen the warnings.
This PR adds a warning data class that contains the category and text of the warning.