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.

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

Already on GitHub? Sign in to your account

fix: UI fixes #2653

Merged
merged 3 commits into from
May 20, 2024
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ export const ActionSettingsTab = ({
This is a code action. Please make changes in your code base.
</p>
) : actionClass.type === "noCode" ? (
<>
<div className="max-h-60 overflow-auto">
<div>
<Label>Select By</Label>
</div>
Expand All @@ -225,7 +225,7 @@ export const ActionSettingsTab = ({
setIsInnerHtml={setIsInnerHtml}
register={register}
/>
</>
</div>
) : actionClass.type === "automatic" ? (
<p className="text-sm text-slate-600">
This action was created automatically. You cannot make changes to it.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,7 @@ export const MainNavigation = ({
{product && (
<aside
className={cn(
"z-20 flex flex-col justify-between rounded-r-xl border-r border-slate-200 bg-white pt-3 shadow-md transition-all duration-100",
"z-40 flex flex-col justify-between rounded-r-xl border-r border-slate-200 bg-white pt-3 shadow-md transition-all duration-100",
!isCollapsed ? "w-sidebar-collapsed" : "w-sidebar-expanded",
environment.type === "development" ? `h-[calc(100vh-1.25rem)]` : "h-screen"
)}>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ interface SideBarProps {

export const TopControlBar = ({ environment, environments }: SideBarProps) => {
return (
<div className="fixed inset-0 top-0 z-10 flex h-14 w-full items-center justify-end bg-slate-50 px-6">
<div className="fixed inset-0 top-0 z-30 flex h-14 w-full items-center justify-end bg-slate-50 px-6">
<div className="shadow-xs z-10">
<div className="flex w-fit space-x-2 py-2">
<WidgetStatusIndicator environment={environment} type="mini" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ export const IndividualInviteTab = ({
</div>
</div>
</div>
<div className="flex justify-end p-6">
<div className="flex justify-end py-6">
<div className="flex space-x-2">
<Button
size="sm"
Expand Down
14 changes: 7 additions & 7 deletions packages/ui/Actions/components/PageUrlSelector.tsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { Label } from "@radix-ui/react-dropdown-menu";
import clsx from "clsx";
import { Control, Controller, UseFormRegister } from "react-hook-form";

import { cn } from "@formbricks/lib/cn";
import { TActionClass } from "@formbricks/types/actionClasses";

import { AdvancedOptionToggle } from "../../AdvancedOptionToggle";
Expand Down Expand Up @@ -42,9 +42,9 @@ export const PageUrlSelector = ({
title="Page URL"
description="If a user visits a specific URL"
childBorder={true}>
<div className="col-span-1 space-y-3 p-4">
<div className="grid grid-cols-3 gap-x-8">
<div className="col-span-1">
<div className="col-span-1 w-full space-y-3 p-4">
<div className="flex w-full items-end gap-2">
<div>
<Label>URL</Label>
<Controller
name="noCodeConfig.pageUrl.rule"
Expand All @@ -66,7 +66,7 @@ export const PageUrlSelector = ({
)}
/>
</div>
<div className="col-span-2 flex items-end">
<div className="flex flex-1 items-end">
<Input
type="text"
className="bg-white"
Expand All @@ -81,7 +81,7 @@ export const PageUrlSelector = ({
Enter a URL to see if a user visiting it would be tracked.
</div>
<div className=" rounded bg-slate-50">
<div className="mt-1 flex">
<div className="mt-1 flex items-end">
<Input
type="text"
value={testUrl}
Expand All @@ -90,7 +90,7 @@ export const PageUrlSelector = ({
setTestUrl(e.target.value);
setIsMatch("default");
}}
className={clsx(
className={cn(
isMatch === "yes"
? "border-green-500 bg-green-50"
: isMatch === "no"
Expand Down
3 changes: 1 addition & 2 deletions packages/ui/Modal/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,7 @@ const DialogContent = React.forwardRef<
`${noPadding ? "" : "px-4 pb-4 pt-5 sm:p-6"}`,
"data-[state='closed']:animate-fadeOut data-[state='open']:animate-fadeIn",
size && sizeClassName && sizeClassName[size],
className,
"max-h-screen overflow-y-auto"
className
)}
{...props}
onPointerDownOutside={(e) => {
Expand Down
2 changes: 1 addition & 1 deletion packages/ui/SingleResponseCard/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -301,7 +301,7 @@ export const SingleResponseCard = ({
<div className={clsx("group relative", isOpen && "min-h-[300px]")}>
<div
className={clsx(
"relative z-30 my-6 rounded-xl border border-slate-200 bg-white shadow-sm transition-all",
"relative z-20 my-6 rounded-xl border border-slate-200 bg-white shadow-sm transition-all",
pageType === "response" &&
(isOpen
? "w-3/4"
Expand Down
Loading