Skip to content
This repository has been archived by the owner on Jul 3, 2023. It is now read-only.

Add file tree component #283

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add file tree component #283

wants to merge 5 commits into from

Conversation

JosephusPaye
Copy link
Member

@JosephusPaye JosephusPaye commented Jul 4, 2019

Continuing the break up of #234 (Script Tuner), this PR adds a file tree component for browsing and selecting a file from a list of files.

Storybook: https://nusight-pr-283.herokuapp.com/storybook/?path=/story/components-file-tree--interactive

@BrendanAnnable BrendanAnnable temporarily deployed to nusight-pr-283 July 4, 2019 04:47 Inactive
@BrendanAnnable BrendanAnnable temporarily deployed to nusight-pr-283 July 4, 2019 05:23 Inactive
@JosephusPaye JosephusPaye requested review from BrendanAnnable and TrentHouliston and removed request for BrendanAnnable February 17, 2020 23:52
Copy link
Member

@TrentHouliston TrentHouliston left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't speak to if there is a better way to do this but the code seems sane and it works.
If you want to know if there is a better way to code parts you'll have to wait for @BrendanAnnable

@BrendanAnnable BrendanAnnable removed their request for review July 3, 2023 16:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants