Skip to content

Commit

Permalink
fix: state icon color on group titles (#1435)
Browse files Browse the repository at this point in the history
  • Loading branch information
aaryan610 authored Jul 4, 2023
1 parent c3fe221 commit 3906503
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 16 deletions.
15 changes: 2 additions & 13 deletions apps/app/components/core/board-view/board-header.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -57,18 +57,6 @@ export const BoardHeader: React.FC<Props> = ({
: null
);

let bgColor = "#000000";
if (selectedGroup === "state") bgColor = currentState?.color ?? "#000000";

if (selectedGroup === "priority")
groupTitle === "high"
? (bgColor = "#dc2626")
: groupTitle === "medium"
? (bgColor = "#f97316")
: groupTitle === "low"
? (bgColor = "#22c55e")
: (bgColor = "#ff0000");

const getGroupTitle = () => {
let title = addSpaceIfCamelCase(groupTitle);

Expand Down Expand Up @@ -96,7 +84,8 @@ export const BoardHeader: React.FC<Props> = ({

switch (selectedGroup) {
case "state":
icon = currentState && getStateGroupIcon(currentState.group, "16", "16", bgColor);
icon =
currentState && getStateGroupIcon(currentState.group, "16", "16", currentState.color);
break;
case "priority":
icon = getPriorityIcon(groupTitle, "text-lg");
Expand Down
5 changes: 2 additions & 3 deletions apps/app/components/core/list-view/single-list.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ import { PROJECT_ISSUE_LABELS, PROJECT_MEMBERS } from "constants/fetch-keys";
type Props = {
type?: "issue" | "cycle" | "module";
currentState?: IState | null;
bgColor?: string;
groupTitle: string;
groupedByIssues: {
[key: string]: IIssue[];
Expand All @@ -53,7 +52,6 @@ type Props = {
export const SingleList: React.FC<Props> = ({
type,
currentState,
bgColor,
groupTitle,
groupedByIssues,
selectedGroup,
Expand Down Expand Up @@ -113,7 +111,8 @@ export const SingleList: React.FC<Props> = ({

switch (selectedGroup) {
case "state":
icon = currentState && getStateGroupIcon(currentState.group, "16", "16", bgColor);
icon =
currentState && getStateGroupIcon(currentState.group, "16", "16", currentState.color);
break;
case "priority":
icon = getPriorityIcon(groupTitle, "text-lg");
Expand Down

1 comment on commit 3906503

@vercel
Copy link

@vercel vercel bot commented on 3906503 Jul 4, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Successfully deployed to the following URLs:

plane-dev – ./apps/app

plane-dev.vercel.app
plane-dev-plane.vercel.app
plane-dev-git-develop-plane.vercel.app

Please sign in to comment.