[BI-895] - Convert the program user's table to use Buefy data tables #100
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The program user's table now uses Buefy's Table component (https://buefy.org/documentation/table/). Rather than implementing Buefy's table directly in
ProgramUsersTable
or modifyingExpandableRowTable
, a new component namedExpandableTable
was created. The main reason for not modifyingExpandableRowTable
was to limit the scope of impact to justProgramUsersTable
rather than the entire UI, and to allow for each table currently using that component to be migrated and tested individually and incrementally.The
ProgramUsersTable
should have feature parity with the original implementation. In addition, columns can now be sorted.All features of Buefy Table should be accessible to client code using
ExpandableTable
, as the$attrs
are passed through to Buefy Table. As an example, Buefy Table allows configuration to indicate if data is loading, butExpandableTable
does not have to have an explicit property defined to set this (ProgramUsersTable
does set theloading
value).Known bugs
ExpandableRowTable
). Issue -> https://breedinginsight.atlassian.net/browse/BI-1035Future Development
ExpandableRowTable
usages toExpandableTable