-
-
Notifications
You must be signed in to change notification settings - Fork 740
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: split projects view into "my projects" and "other projects" (#6886
) This PR removes the previous "my projects" filter in favor always splitting projects, but showing both on the main screen. To make it a bit easier to work with, it also moves the project group component into its own file, causing some extra lines of code change. My apologies 🙇🏼
- Loading branch information
1 parent
b8380a0
commit b6833d9
Showing
6 changed files
with
207 additions
and
204 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
138 changes: 138 additions & 0 deletions
138
frontend/src/component/project/ProjectList/ProjectGroup.tsx
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,138 @@ | ||
| import { Link } from 'react-router-dom'; | ||
| import { ConditionallyRender } from 'component/common/ConditionallyRender/ConditionallyRender'; | ||
| import { ProjectCard as LegacyProjectCard } from '../ProjectCard/ProjectCard'; | ||
| import { ProjectCard as NewProjectCard } from '../NewProjectCard/NewProjectCard'; | ||
| import type { IProjectCard } from 'interfaces/project'; | ||
| import loadingData from './loadingData'; | ||
| import { TablePlaceholder } from 'component/common/Table'; | ||
| import { styled, Typography } from '@mui/material'; | ||
| import { useUiFlag } from 'hooks/useUiFlag'; | ||
|
|
||
| const StyledProjectGroupContainer = styled('article')(({ theme }) => ({ | ||
| h3: { | ||
| marginBlockEnd: theme.spacing(2), | ||
| }, | ||
|
|
||
| '&+&': { | ||
| marginBlockStart: theme.spacing(4), | ||
| }, | ||
| })); | ||
|
|
||
| /** | ||
| * @deprecated Remove after with `projectsListNewCards` flag | ||
| */ | ||
| const StyledDivContainer = styled('div')(({ theme }) => ({ | ||
| display: 'flex', | ||
| flexWrap: 'wrap', | ||
| [theme.breakpoints.down('sm')]: { | ||
| justifyContent: 'center', | ||
| }, | ||
| })); | ||
|
|
||
| const StyledGridContainer = styled('div')(({ theme }) => ({ | ||
| display: 'grid', | ||
| gridTemplateColumns: 'repeat(auto-fill, minmax(250px, 1fr))', | ||
| gap: theme.spacing(2), | ||
| })); | ||
|
|
||
| const StyledCardLink = styled(Link)(({ theme }) => ({ | ||
| color: 'inherit', | ||
| textDecoration: 'none', | ||
| border: 'none', | ||
| padding: '0', | ||
| background: 'transparent', | ||
| fontFamily: theme.typography.fontFamily, | ||
| pointer: 'cursor', | ||
| })); | ||
|
|
||
| export const ProjectGroup: React.FC<{ | ||
| sectionTitle?: string; | ||
| projects: IProjectCard[]; | ||
| loading: boolean; | ||
| searchValue: string; | ||
| handleHover: (id: string) => void; | ||
| }> = ({ sectionTitle, projects, loading, searchValue, handleHover }) => { | ||
| const useNewProjectCards = useUiFlag('projectsListNewCards'); | ||
|
|
||
| const [StyledItemsContainer, ProjectCard] = useNewProjectCards | ||
| ? [StyledGridContainer, NewProjectCard] | ||
| : [StyledDivContainer, LegacyProjectCard]; | ||
|
|
||
| return ( | ||
| <StyledProjectGroupContainer> | ||
| <ConditionallyRender | ||
| condition={Boolean(sectionTitle)} | ||
| show={<Typography component='h3'>{sectionTitle}</Typography>} | ||
| /> | ||
| <ConditionallyRender | ||
| condition={projects.length < 1 && !loading} | ||
| show={ | ||
| <ConditionallyRender | ||
| condition={searchValue?.length > 0} | ||
| show={ | ||
| <TablePlaceholder> | ||
| No projects found matching “ | ||
| {searchValue} | ||
| ” | ||
| </TablePlaceholder> | ||
| } | ||
| elseShow={ | ||
| <TablePlaceholder> | ||
| No projects available. | ||
| </TablePlaceholder> | ||
| } | ||
| /> | ||
| } | ||
| elseShow={ | ||
| <StyledItemsContainer> | ||
| <ConditionallyRender | ||
| condition={loading} | ||
| show={() => | ||
| loadingData.map((project: IProjectCard) => ( | ||
| <ProjectCard | ||
| data-loading | ||
| onHover={() => {}} | ||
| key={project.id} | ||
| name={project.name} | ||
| id={project.id} | ||
| mode={project.mode} | ||
| memberCount={2} | ||
| health={95} | ||
| featureCount={4} | ||
| /> | ||
| )) | ||
| } | ||
| elseShow={() => ( | ||
| <> | ||
| {projects.map((project: IProjectCard) => ( | ||
| <StyledCardLink | ||
| key={project.id} | ||
| to={`/projects/${project.id}`} | ||
| > | ||
| <ProjectCard | ||
| onHover={() => | ||
| handleHover(project.id) | ||
| } | ||
| name={project.name} | ||
| mode={project.mode} | ||
| memberCount={ | ||
| project.memberCount ?? 0 | ||
| } | ||
| health={project.health} | ||
| id={project.id} | ||
| featureCount={ | ||
| project.featureCount | ||
| } | ||
| isFavorite={project.favorite} | ||
| /> | ||
| </StyledCardLink> | ||
| ))} | ||
| </> | ||
| )} | ||
| /> | ||
| </StyledItemsContainer> | ||
| } | ||
| /> | ||
| </StyledProjectGroupContainer> | ||
| ); | ||
| }; |
Oops, something went wrong.