Split AlertTable into smaller components#2732

Merged

robertbrignull merged 23 commits intomainfrom

robertbrignull/AlertTable-decompose

Aug 22, 2023

Commits on Aug 21, 2023

Move AlertTableNoResults to a new component in a new file
robertbrignull
committed
Move AlertTableTruncatedMessage to a new component in a new file
robertbrignull
committed
Construct rows using maps instead of mutating array from multiple places
robertbrignull
committed
Pull out AlertTablePathNodeRow to a separate component
robertbrignull
committed
Inline some rendering in AlertTablePathNodeRow
robertbrignull
committed
Use useMemo for callback passed to SarifLocation.onClick in AlertTablePathNodeRow
robertbrignull
committed
Pull out AlertTablePathRow to a separate component
robertbrignull
committed
Use useMemo for callback passed to AlertTableDropdownIndicatorCell.onClick in AlertTablePathRow
robertbrignull
committed
Do all rendering in AlertTablePathRow in one place
robertbrignull
committed
Pull out AlertTableResultRow to a separate component
robertbrignull
committed
Use useMemo for callback passed to AlertTableDropdownIndicatorCell.onClick in AlertTableResultRow
robertbrignull
committed
Simplify rendering in AlertTable
robertbrignull
committed
Teach AlertTableResultRow how to render results without codeFlows
robertbrignull
committed
Move rendering of locationCells to inside AlertTableResuiltRow
robertbrignull
committed
Use useMemo for callback passed to SarifLocation.onClick in AlertTableResultRow
robertbrignull
committed
Move rendering of msg to inside AlertTableResuiltRow
robertbrignull
committed
Move rendering of AlertTablePathRow to inside of AlertTableResultRow
robertbrignull
committed
Remove rows array and render AlertTableResultRow inline
robertbrignull
committed
Pass around toggle instead of toggler
robertbrignull
committed
Simplify passing similar props
robertbrignull
committed
Use useCallback when calling toggleExpanded
robertbrignull
committed
Convert updateSelectionCallback to not use double-nested methods
robertbrignull
committed

Commits on Aug 22, 2023

Merge branch 'main' into robertbrignull/AlertTable-decompose
robertbrignull
authored

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Split AlertTable into smaller components#2732

Split AlertTable into smaller components#2732
robertbrignull merged 23 commits intomainfrom
robertbrignull/AlertTable-decompose

Commits on Aug 21, 2023

Move AlertTableNoResults to a new component in a new file

Move AlertTableTruncatedMessage to a new component in a new file

Construct rows using maps instead of mutating array from multiple places

Pull out AlertTablePathNodeRow to a separate component

Inline some rendering in AlertTablePathNodeRow

Use useMemo for callback passed to SarifLocation.onClick in AlertTablePathNodeRow

Pull out AlertTablePathRow to a separate component

Use useMemo for callback passed to AlertTableDropdownIndicatorCell.onClick in AlertTablePathRow

Do all rendering in AlertTablePathRow in one place

Pull out AlertTableResultRow to a separate component

Use useMemo for callback passed to AlertTableDropdownIndicatorCell.onClick in AlertTableResultRow

Simplify rendering in AlertTable

Teach AlertTableResultRow how to render results without codeFlows

Move rendering of locationCells to inside AlertTableResuiltRow

Use useMemo for callback passed to SarifLocation.onClick in AlertTableResultRow

Move rendering of msg to inside AlertTableResuiltRow

Move rendering of AlertTablePathRow to inside of AlertTableResultRow

Remove rows array and render AlertTableResultRow inline

Pass around toggle instead of toggler

Simplify passing similar props

Use useCallback when calling toggleExpanded

Convert updateSelectionCallback to not use double-nested methods

Commits on Aug 22, 2023

Merge branch 'main' into robertbrignull/AlertTable-decompose