Mar 11, 2024 · Mar 8, 2024
diff --git a/site/src/modules/resources/PortForwardButton.tsx b/site/src/modules/resources/PortForwardButton.tsx
  const canSharePortsPublic =
    canSharePorts && template.max_port_share_level === "public";

  const publicMenuItem = (
    <>
      {canSharePortsPublic ? (
        <MenuItem value="public">Public</MenuItem>
      ) : (
        <Tooltip title="This workspace template does not allow sharing ports with unauthenticated users.">
          {/* Tooltips don't work directly on disabled MenuItem components so you must wrap in div. */}
          <div>
            <MenuItem value="public" disabled>
              Public
            </MenuItem>
          </div>
        </Tooltip>
      )}
    </>
  const disabledPublicMenuItem = (
    <Tooltip title="This workspace template does not allow sharing ports with unauthenticated users.">
      {/* Tooltips don't work directly on disabled MenuItem components so you must wrap in div. */}
      <div>
        <MenuItem value="public" disabled>
          Public
        </MenuItem>
      </div>
    </Tooltip>
  );

  return (
                          <MenuItem value="authenticated">
                            Authenticated
                          </MenuItem>
                          {publicMenuItem}
                          {canSharePortsPublic ? (
                            <MenuItem value="public">Public</MenuItem>
                          ) : (
                            disabledPublicMenuItem
                          )}
                        </Select>
                      </FormControl>
                      <Button
                    label="Sharing Level"
                  >
                    <MenuItem value="authenticated">Authenticated</MenuItem>
                    {publicMenuItem}
                    {canSharePortsPublic ? (
                      <MenuItem value="public">Public</MenuItem>
                    ) : (
                      disabledPublicMenuItem
                    )}
                  </TextField>
                  <LoadingButton
                    variant="contained"
Original file line number	Diff line number	Diff line change
Expand Up		@@ -205,21 +205,15 @@ export const PortForwardPopoverView: FC<PortForwardPopoverViewProps> = ({
		const canSharePortsPublic =
		canSharePorts && template.max_port_share_level === "public";

		const publicMenuItem = (
		<>
		{canSharePortsPublic ? (
		<MenuItem value="public">Public</MenuItem>
		) : (
		<Tooltip title="This workspace template does not allow sharing ports with unauthenticated users.">
		{/* Tooltips don't work directly on disabled MenuItem components so you must wrap in div. */}
		<div>
		<MenuItem value="public" disabled>
		Public
		</MenuItem>
		</div>
		</Tooltip>
		)}
		</>
		const disabledPublicMenuItem = (
		<Tooltip title="This workspace template does not allow sharing ports with unauthenticated users.">
		{/* Tooltips don't work directly on disabled MenuItem components so you must wrap in div. */}
		<div>
		<MenuItem value="public" disabled>
		Public
		</MenuItem>
		</div>
		</Tooltip>
		);

		return (
Expand DownExpand Up		@@ -447,7 +441,11 @@ export const PortForwardPopoverView: FC<PortForwardPopoverViewProps> = ({
		<MenuItem value="authenticated">
		Authenticated
		</MenuItem>
		{publicMenuItem}
		{canSharePortsPublic ? (
		<MenuItem value="public">Public</MenuItem>
		) : (
		disabledPublicMenuItem
		)}
		</Select>
		</FormControl>
		<Button
Expand DownExpand Up		@@ -512,7 +510,11 @@ export const PortForwardPopoverView: FC<PortForwardPopoverViewProps> = ({
		label="Sharing Level"
		>
		<MenuItem value="authenticated">Authenticated</MenuItem>
		{publicMenuItem}
		{canSharePortsPublic ? (
		<MenuItem value="public">Public</MenuItem>
		) : (
		disabledPublicMenuItem
		)}
		</TextField>
		<LoadingButton
		variant="contained"
Expand Down