Paragon table deprecation migration to DataTable

[abdullahwaheed]

Ticket

Migrate off deprecated Paragon components

What has changed

Removed deprecated Table component of paragon and updated it with DataTable

References

Paragon Table
Paragon DataTable

[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]

@abdullahwaheed I don't think the changes are playing nice when I tested it locally. All the fields of the ProgramRecord component are displaying vertically rather than horizontally as part of the table. A screenshot can be seen here:

Versus how it looked before:

[justinhynes]

Moving to using the DataTable component appears to have broken the ProgramRecord table. Please see comment and screenshot.

[abdullahwaheed]

@abdullahwaheed I don't think the changes are playing nice when I tested it locally. All the fields of the ProgramRecord compoenent are displying vertically then horizontally as part of the table. A screenshot can be seen here:

will test it locally and make the required changes

[abdullahwaheed]

@justinhynes it working fine on local env, screenshot is attached. Please pull latest code and check. if the issue persists, could you please provide the testing env you are using?

[justinhynes]

@abdullahwaheed I'll check it again this morning and get back to you. Thanks. I was testing this with a local devstack instance and using the edx.org theme.

[justinhynes]

I tested this again and I think I have narrowed this down to an issue with the edx.org theme specifically.

I pulled this branch, ran make static and then attempted to access a ProgramRecord example.

I switched to the openedx theme and see:

After switching to the edx.org theme:

[abdullahwaheed]

i think there are some conflicting styles there, let me check this with edx.org theme as well

[abdullahwaheed]

@justinhynes could you please test it again

[justinhynes]

@abdullahwaheed Sorry! I missed this message. I took another look this morning and it looks good!