Skip to content

Commit 11781b7

Browse files
refactor: update background layers and layout adjustments in dropdown and button components for improved UI consistency
1 parent c975190 commit 11781b7

File tree

4 files changed

+14
-10
lines changed

4 files changed

+14
-10
lines changed

apps/web/app/(all)/[workspaceSlug]/(projects)/star-us-link.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ export function StarUsOnGitHubLink() {
2929
},
3030
})
3131
}
32-
className="flex flex-shrink-0 items-center gap-1.5 rounded-sm bg-layer-1 px-3 py-1.5"
32+
className="flex flex-shrink-0 items-center gap-1.5 rounded-sm bg-layer-2 px-3 py-1.5"
3333
href="https://github.com/makeplane/plane"
3434
target="_blank"
3535
rel="noopener noreferrer"

apps/web/core/components/dropdowns/buttons.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ function BackgroundButton(props: ButtonProps) {
104104
variant="ghost"
105105
size="sm"
106106
className={cn(
107-
"h-full w-full flex items-center justify-start gap-1.5 bg-layer-3 hover:bg-layer-1-hover",
107+
"h-full w-full flex items-center justify-between gap-1.5 bg-layer-3 hover:bg-layer-1-hover",
108108
className
109109
)}
110110
>
@@ -129,7 +129,7 @@ function TransparentButton(props: ButtonProps) {
129129
variant="ghost"
130130
size="sm"
131131
className={cn(
132-
"h-full w-full flex items-center justify-start gap-1.5",
132+
"h-full w-full flex items-center justify-between gap-1.5",
133133
{
134134
"bg-layer-transparent-active": isActive,
135135
},

apps/web/core/components/dropdowns/estimate.tsx

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -192,12 +192,16 @@ export const EstimateDropdown = observer(function EstimateDropdown(props: Props)
192192
>
193193
{!hideIcon && <EstimatePropertyIcon className="h-3 w-3 flex-shrink-0" />}
194194
{(selectedEstimate || placeholder) && BUTTON_VARIANTS_WITH_TEXT.includes(buttonVariant) && (
195-
<span className="flex-grow truncate">
196-
{selectedEstimate
197-
? currentActiveEstimate?.type === EEstimateSystem.TIME
198-
? convertMinutesToHoursMinutesString(Number(selectedEstimate.value))
199-
: selectedEstimate.value
200-
: placeholder}
195+
<span className="truncate">
196+
{selectedEstimate ? (
197+
currentActiveEstimate?.type === EEstimateSystem.TIME ? (
198+
convertMinutesToHoursMinutesString(Number(selectedEstimate.value))
199+
) : (
200+
selectedEstimate.value
201+
)
202+
) : (
203+
<span className="text-placeholder">{placeholder}</span>
204+
)}
201205
</span>
202206
)}
203207
{dropdownArrow && (

apps/web/core/components/issues/issue-detail/module-select.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ export const IssueModuleSelect = observer(function IssueModuleSelect(props: TIss
6464
placeholder={t("module.no_module")}
6565
disabled={disableSelect}
6666
className="group h-full w-full"
67-
buttonContainerClassName="w-full text-left h-7.5 rounded-sm"
67+
buttonContainerClassName="w-full text-left rounded-sm"
6868
buttonClassName={`text-body-xs-medium justify-between ${issue?.module_ids?.length ? "" : "text-placeholder"}`}
6969
buttonVariant="transparent-with-text"
7070
hideIcon

0 commit comments

Comments
 (0)