NETOBSERV-608 UI: Reorder columns in flow table

[jpinsonneau]

Add table drag & drop capabilities

Screencast_21_11_2022_14.01.06.webm

[andrew-ronaldson]

@jpinsonneau Can this be contributed to console as well?
Is this still using the PF table?

[jpinsonneau]

@jpinsonneau Can this be contributed to console as well? Is this still using the PF table?

Yes ! However we are using TableComposable whereas console uses legacy one.

I'll take a look at react-table lib to see if we can contribute in both implementations.

[stleerh]

I did not run this code, so what happens when you resize a column and then add a new column?  Previously, all the columns were in some predefined order, so where will the new column get added?

[jpinsonneau]

I did not run this code, so what happens when you resize a column and then add a new column?  Previously, all the columns were in some predefined order, so where will the new column get added?

Reordering from table should also reorder columns in the popup and vice versa.
Custom sizing is kept for the columns you touched, the others just load with their default width.

[jpinsonneau]

@stleerh just fixed a bug while testing your case => 3e15f2e
The not selected columns disapeared in the popup after reorder in the table.

To solve that, I move all not selected columns at the end of the list:
Screencast_23_11_2022_10:04:40.webm

[Amoghrd]

/ok-to-test

[github-actions]

New image: ["quay.io/netobserv/network-observability-console-plugin:3545dbd"]. It will expire after two weeks.

[jotak]

/lgtm
(code review - not tested)

[jpinsonneau]

/approve thanks @stleerh & @jotak

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jpinsonneau

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [jpinsonneau]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment