Paragon StatusAlert deprecation

[abdullahwaheed]

Ticket

Migrate off deprecated Paragon components

What has changed

Removed deprecated StatusAlert component of paragon and updated it with Alert

References

Paragon StatusAlert
Paragon Alert

[davidjoy]

This seems like a fine change - the owning team should also review so they're aware of the change, it should be tested by Fed-BOM and we can get it in. I did a cursory review.

[justinhynes]

Screenshots before and after the change...

Before

After

[justinhynes]

Changes look good, I checked it locally and noticed that the Dismiss button text seems larger than expected. It seems very large compared to the examples from Paragon. Is there anyway to adjust that?

[justinhynes]

@abdullahwaheed -- quick question on the size of the Dismiss button before accepting this change. Thanks for the help and let me know what you think!

[abdullahwaheed]

@justinhynes .btn styling was affecting its font, removed this to make it consistent with library

[justinhynes]

I think this change is also having conflicts with the edx.org theme. :(

In the openedx theme the button size looks right:

In the edx.org theme the button size is unchanged:

Sorry :(

[justinhynes]

@abdullahwaheed Is this ready for another look too? Or is this still being worked on?

[abdullahwaheed]

@justinhynes need some changes from #1691. Will update it once its merged

[abdullahwaheed]

@justinhynes its ready for review, kindly test it again

[justinhynes]

@abdullahwaheed I am still seeing a button size discrepancy between the openedx and edx.org themes:

openedx:

edx.org:

[abdullahwaheed]

I have tested it on edx.org theme and override styles are working.

[justinhynes]

@abdullahwaheed Thanks for the update. Something might not quite be right in my local environment. Based on your screenshots I think this is good to go.