Skip to content

Commit

Permalink
Added all-around margins for both error and warning notifications
Browse files Browse the repository at this point in the history
  • Loading branch information
cdhenley219 committed Oct 30, 2024
1 parent f9cb7dc commit ffe3a6b
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/components/Trends/TrendsPanel/TrendsPanel.scss
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
width: 100%;

.m-notification {
margin: 10px;
margin-bottom: 10px;
}
}
&.external-tooltip {
Expand Down
3 changes: 2 additions & 1 deletion src/components/Warnings/Error.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,11 @@
import getIcon from '../iconMap';
import PropTypes from 'prop-types';
import './Error.scss';

export const ErrorBlock = ({ text }) => (
<div
role="alert"
className="m-notification m-notification--visible m-notification--error"
className="error m-notification m-notification--visible m-notification--error"
>
{getIcon('error-round')}
<div className="m-notification__content">
Expand Down
5 changes: 5 additions & 0 deletions src/components/Warnings/Error.scss
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
.error {
&.m-notification {
margin: 10px;
}
}
4 changes: 4 additions & 0 deletions src/components/Warnings/Warning.scss
Original file line number Diff line number Diff line change
@@ -1,4 +1,8 @@
.warning {
&.m-notification {
margin: 10px;
}

.close {
position: absolute;
right: 0;
Expand Down

0 comments on commit ffe3a6b

Please sign in to comment.