Skip to content

Commit

Permalink
PR fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
ksalin committed Sep 11, 2023
1 parent e7acea8 commit 03581a3
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 7 deletions.
2 changes: 1 addition & 1 deletion src/components/Notification/Notification.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ export const Notification = ({
duration &&
setTimeout(() => {
setHidden(true);
deleteNotification && deleteNotification(index || -1);
deleteNotification && deleteNotification(index || 0);
}, duration);

return (
Expand Down
9 changes: 3 additions & 6 deletions src/components/NotificationCarousel/NotificationCarousel.tsx
Original file line number Diff line number Diff line change
@@ -1,19 +1,16 @@
import React, { useState } from 'react';
import styled from 'styled-components';
import { ComponentBaseProps, generateRandomString } from '../../shared';
import { generateRandomString } from '../../shared';
import { Notification } from '../Notification/Notification';
import { Wrapper, WrapperProps } from '../Wrapper';
import { notificationSpacing } from './styles';

/**
* Notification carousel component properties
* Extends html label attributes
* https://developer.mozilla.org/en-US/docs/Web/HTML/Element/label#attributes
*/
export interface NotificationCarouselProps
extends WrapperProps,
export interface NotificationCarouselProps extends WrapperProps /*,
ComponentBaseProps<HTMLLabelElement>,
React.LabelHTMLAttributes<HTMLLabelElement> {
React.LabelHTMLAttributes<HTMLLabelElement>*/ {
/**
* List of items (notifications) to stack to each other
*/
Expand Down

0 comments on commit 03581a3

Please sign in to comment.