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.
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
Projects page cleanup #33117
Projects page cleanup #33117
Conversation
limitedGallery={projectsData.limitedGallery} | ||
canShare={projectsData.canShare} | ||
/> | ||
{/* TODO: Move components below into <PersonalProjectsTable/>? */} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those Dialogs are only relevant from the Personal Project Table, so sounds good to me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lol i almost forgot about this! i'll make a follow-up PR to scooch these over 🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yay! Thanks for taking the time to refactor this and all of the little clean up along the way. Lovely!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not very familiar with this area yet, but this makes sense to me and seems cleaner!
This is a pure refactor, so there should be no user-visible changes here.
Both
projects/index.html.haml
andprojects/index.js
had a lot of artifacts from our AngularJS days, so I'm cleaning that up here to simplify the code (end goal is to add a "My Libraries" tab to the projects/ page).Follow-up work:
<GallerySwitcher/>
to<ProjectsGallery/>
now that this component renders both the gallery tabs and the content -- Rename <GallerySwitcher/> to <ProjectsGallery/> #33121<PublishDialog/>
and<DeleteProjectDialog/>
into<PersonalProjectsTable/>
Reviewer Checklist: