Skip to content

Commit d2872f5

Browse files
adding create project modal (#18871)
* adding create project modal * setting up portal w/ popover for dropdown * Fixing issues with trying to display dropdown in a portal * cleanup and adding filling in create project modal a bit * Adding some git auth functionality * add project id for now * remove github logic that checks all repos * reload projects when we create a new one * fixing repo sorting to handle multiple projects better * fill in repo name for projects * excluding projects in repo results * handling all new project buttons * refactor Dropdown2 to use radix popover * use radix css var for width * remove project distinction for now * maintain focus on search input * adjusting validation logic on repo when creating a project * remove scmService from server impl * set project name on backend for now * make w-full again * simplify * implement hasReadAccess for BBS * remove canInstallWebhook and GitHub app check during webhook installation * moving project validation into project service * simplify * ui cleanup * handle case where there is no repoProvider * mock repositoryProvider.hasReadAccess * missed a Project.slug removal * ui cleanup per feedback * adding script to start on other port for local fe dev flow
1 parent a1657f7 commit d2872f5

24 files changed

+754
-413
lines changed

components/dashboard/package.json

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
"@connectrpc/connect": "1.1.2",
99
"@gitpod/gitpod-protocol": "0.1.5",
1010
"@gitpod/public-api": "0.1.5",
11+
"@radix-ui/react-popover": "^1.0.7",
1112
"@stripe/react-stripe-js": "^1.7.2",
1213
"@stripe/stripe-js": "^1.29.0",
1314
"@tanstack/query-async-storage-persister": "^4.29.19",
@@ -84,6 +85,7 @@
8485
},
8586
"scripts": {
8687
"start": "BROWSER=none HMR_HOST=`gp url 3001` craco start",
88+
"start-local": "BROWSER=none HMR_HOST=`gp url 3000` craco start",
8789
"build": "craco build --verbose",
8890
"lint": "eslint --max-warnings=0 --ext=.jsx,.js,.tsx,.ts ./src",
8991
"test": "yarn test:unit",

components/dashboard/src/components/Alert.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ export default function Alert(props: AlertProps) {
111111
return (
112112
<div
113113
className={classNames(
114-
"flex relative whitespace-pre-wrap p-4",
114+
"flex items-center relative whitespace-pre-wrap p-4",
115115
info.txtCls,
116116
props.className,
117117
light ? "" : info.bgCls,
@@ -121,15 +121,15 @@ export default function Alert(props: AlertProps) {
121121
{showIcon && <span className={`mt-1 mr-4 h-4 w-4 ${info.iconColor}`}>{props.icon ?? info.icon}</span>}
122122
<span className="flex-1 text-left">{props.children}</span>
123123
{props.closable && (
124-
<span className={`mt-1 ml-4`}>
124+
<span className={`ml-4`}>
125125
{/* Use an IconButton component once we make it */}
126126
<Button
127127
type="secondary"
128128
className="bg-transparent hover:bg-transparent"
129129
onClick={handleClose}
130130
autoFocus={autoFocusClose}
131131
>
132-
<XSvg className="w-3 h-4 cursor-pointer" />
132+
<XSvg className="w-3 h-4 cursor-pointer text-white" />
133133
</Button>
134134
</span>
135135
)}

components/dashboard/src/components/AuthorizeGit.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,12 +53,7 @@ export const AuthorizeGit: FC<{ className?: string }> = ({ className }) => {
5353
const verifiedProviders = authProviders.data.filter((ap) => ap.verified);
5454

5555
return (
56-
<div
57-
className={classNames(
58-
"w-full text-center border-2 border-gray-100 dark:border-gray-800 p-4 m-4 rounded-lg py-10",
59-
className,
60-
)}
61-
>
56+
<div className={classNames("text-center p-4 m-4 py-10", className)}>
6257
{verifiedProviders.length === 0 ? (
6358
<>
6459
<Heading3 className="pb-2">No Git integrations</Heading3>

0 commit comments

Comments
 (0)