Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Consider putting the sidebar on the main camera screen into its own file #90

Open
jbphet opened this issue May 8, 2023 · 0 comments
Open

Consider putting the sidebar on the main camera screen into its own file #90

jbphet opened this issue May 8, 2023 · 0 comments
Labels
dev:enhancement New feature or request good first issue Good for newcomers

Comments

@jbphet
Copy link
Contributor

jbphet commented May 8, 2023

As of this writing, CameraMain.js is over 1000 lines long, which is a big module. Around 360 of those lines are the sidebar. I think we should consider putting the sidebar into a separate file to improve modularity.
@jbphet jbphet added the type:bug Something isn't working label May 8, 2023
@jessegreenberg jessegreenberg added dev:enhancement New feature or request and removed type:bug Something isn't working labels May 10, 2023
@brettfiedler brettfiedler added the good first issue Good for newcomers label Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dev:enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jbphet @jessegreenberg @brettfiedler