Skip to content

Commit

Permalink
[WEB 2418] Fix minor UI inconsistencies (#5568)
Browse files Browse the repository at this point in the history
* fix: project features modal padding

* fix: minor ui inconsistencies

* fix: lint issue
  • Loading branch information
gakshita authored Sep 10, 2024
1 parent 7c77fc1 commit 5eb868e
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 4 deletions.
2 changes: 1 addition & 1 deletion web/core/components/cycles/list/root.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ export const CyclesList: FC<ICyclesList> = observer((props) => {
const { completedCycleIds, upcomingCycleIds, cycleIds, workspaceSlug, projectId, isArchived = false } = props;

return (
<ContentWrapper variant={ERowVariant.HUGGING}>
<ContentWrapper variant={ERowVariant.HUGGING} className="flex-row">
<ListLayout>
{isArchived ? (
<>
Expand Down
12 changes: 10 additions & 2 deletions web/core/components/dropdowns/module/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ type Props = TDropdownProps & {
showCount?: boolean;
onClose?: () => void;
renderByDefault?: boolean;
itemClassName?: string;
} & (
| {
multiple: false;
Expand All @@ -51,6 +52,7 @@ type ButtonContentProps = {
showCount: boolean;
showTooltip?: boolean;
value: string | string[] | null;
className?: string;
};

const ButtonContent: React.FC<ButtonContentProps> = (props) => {
Expand All @@ -65,6 +67,7 @@ const ButtonContent: React.FC<ButtonContentProps> = (props) => {
showCount,
showTooltip = false,
value,
className,
} = props;
// store hooks
const { getModuleById } = useModule();
Expand All @@ -87,13 +90,16 @@ const ButtonContent: React.FC<ButtonContentProps> = (props) => {
)}
</div>
) : value.length > 0 ? (
<div className="flex max-w-full flex-grow flex-wrap items-center gap-2 truncate py-0.5">
<div className="flex max-w-full flex-grow flex-wrap items-center gap-2 truncate py-0.5 ">
{value.map((moduleId) => {
const moduleDetails = getModuleById(moduleId);
return (
<div
key={moduleId}
className="flex max-w-full items-center gap-1 rounded bg-custom-background-80 py-1 text-custom-text-200"
className={cn(
"flex max-w-full items-center gap-1 rounded bg-custom-background-80 py-1 text-custom-text-200",
className
)}
>
{!hideIcon && <DiceIcon className="h-2.5 w-2.5 flex-shrink-0" />}
{!hideText && (
Expand Down Expand Up @@ -159,6 +165,7 @@ export const ModuleDropdown: React.FC<Props> = observer((props) => {
const {
button,
buttonClassName,
itemClassName = "",
buttonContainerClassName,
buttonVariant,
className = "",
Expand Down Expand Up @@ -270,6 +277,7 @@ export const ModuleDropdown: React.FC<Props> = observer((props) => {
showTooltip={showTooltip}
value={value}
onChange={onChange as any}
className={itemClassName}
/>
</DropdownButton>
</button>
Expand Down
1 change: 1 addition & 0 deletions web/core/components/issues/issue-detail/module-select.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ export const IssueModuleSelect: React.FC<TIssueModuleSelect> = observer((props)
dropdownArrow
dropdownArrowClassName="h-3.5 w-3.5 hidden group-hover:inline"
multiple
itemClassName="px-2"
/>
</div>
);
Expand Down
2 changes: 1 addition & 1 deletion web/core/store/issue/issue.store.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import update from "lodash/update";
import isEmpty from "lodash/isEmpty";
import set from "lodash/set";
import update from "lodash/update";
import { action, makeObservable, observable, runInAction } from "mobx";
import { computedFn } from "mobx-utils";
// types
Expand Down

0 comments on commit 5eb868e

Please sign in to comment.