Improvements Alert Component

[ryuhangyeong]

Etc

Thanks for making such a great library!

Suggest

Since the close button belongs to the title, it can be displayed only if the title exists. But I think the close button should be outside the title. What do you think?

{title && (
  <div className={classes.title}>
    <span id={titleId} className={classes.label}>
      {title}
    </span>

    {withCloseButton && (
      <CloseButton
        className={classes.closeButton}
        onClick={onClose}
        variant="transparent"
        size={16}
        iconSize={16}
        aria-label={closeButtonLabel}
      />
    )}
  </div>
)}

If you set an empty value to the title, you can create what you want, but I think the part marked in red is awkward.

[zhulien-ivanov]

This was covered here: #1348.

[ryuhangyeong]

Thanks~

[rtivital]

Fixed in 5.1.4

[ryuhangyeong]

Thanks~