influxdata / influxdata/giraffe

Visualization: Table: re-port from UI codebase without architectural changes

Open
#824 0 comments 0 reactions 0 assignees View on GitHub
kind/tech debt team/dumplings team/ui
Dominant language
TypeScript
Stars
189
Forks
33
Avg merge
3d 15h
Merged PRs (30d)
4

Description

There are a couple of nagging issues with Table that should be fixed. As one of the fundamental visualizations, there shouldn't be any bugs associated with Table. And even though there may be pending design and product changes for Table, the existing stuff should work.

### Proposal
Instead of porting the Table visualization code from the UI with changes (by attempting to use functional components rather than class components), the port should just be exactly as-is with only minimal and necessary changes. Also, any minor bugs should be fixed. But none of this should be done by changing the underlying code architect such as moving from class components to functional components. The differences between these two implementations may be the reason for the nagging issues.

Specifically, by doing the above, we are consolidating and hoping to address the following:
https://github.com/influxdata/giraffe/issues/788
https://github.com/influxdata/ui/issues/6147

Contributor guide

Open the contributing guide

Research direction

Start by comparing the current Table visualization with its UI codebase counterpart, then review the related reports in issues 788 and UI issue 6147. The work is done when the port matches the existing UI implementation with only necessary changes and the identified Table bugs are fixed without an architectural rewrite.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
data-visualization, frontend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.