Skip to content

[FIX] grid-template-columns 문자열 연결 시 공백 누락으로 인한 레이아웃 깨짐 수정#425

Merged
kanghaeun merged 3 commits intodevelopfrom
feat/dashboard-column-update
Feb 8, 2026
Merged

[FIX] grid-template-columns 문자열 연결 시 공백 누락으로 인한 레이아웃 깨짐 수정#425
kanghaeun merged 3 commits intodevelopfrom
feat/dashboard-column-update

Conversation

@kanghaeun
Copy link
Contributor

📝작업 내용

grid-template-columns 문자열 연결 시 공백이 누락되어 대시보드 칼럼이 세로로 나열되는 문제가 발생

문자열 연결 시 공백을 추가하여 정상적인 그리드 레이아웃이 적용되도록 수정했습니다.

수정 전

  grid-template-columns: ${({ hasInterview }) =>
    '1fr 1fr 1fr 1.2fr 1.5fr 1fr' + (hasInterview ? '1.2fr' : '')};

수정 후

  grid-template-columns: ${({ hasInterview }) =>
    '1fr 1fr 1fr 1.2fr 1.5fr 1fr' + (hasInterview ? ' 1.2fr' : '')}; // 공백 추가

@kanghaeun kanghaeun self-assigned this Feb 8, 2026
@kanghaeun kanghaeun added the 🐛 bug 버그 이슈 label Feb 8, 2026
@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @kanghaeun, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 대시보드에서 지원자 목록의 레이아웃 문제를 해결하고 인터뷰 시간 표시의 일관성을 개선합니다. grid-template-columns CSS 속성 생성 시 문자열 연결 오류로 인해 발생하던 레이아웃 깨짐 현상을 수정하고, 인터뷰가 필요한 지원자의 경우 승인 상태와 관계없이 인터뷰 시간 필드에 일관된 UI를 제공합니다.

Highlights

  • 그리드 레이아웃 공백 누락 수정: 대시보드 지원자 목록의 grid-template-columns 속성 문자열 연결 시 누락된 공백을 추가하여 레이아웃 깨짐 현상을 수정했습니다.
  • 인터뷰 시간 표시 로직 개선: 인터뷰가 필요한 지원자의 경우, 승인 상태와 관계없이 인터뷰 시간 필드에 일관된 UI(하이픈 표시)를 제공하도록 조건부 렌더링 로직을 개선했습니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog
  • src/pages/admin/Dashboard/components/ApplicantListSection/List/ApplicantList/index.styled.ts
    • grid-template-columns 속성 정의에서 hasInterview 조건부 문자열 연결 시 공백이 누락되어 레이아웃이 깨지는 문제를 수정했습니다.
    • 미디어 쿼리 내부의 grid-template-columns 정의에서도 동일한 공백 누락 문제를 해결했습니다.
  • src/pages/admin/Dashboard/components/ApplicantListSection/List/ApplicantListItem/index.tsx
    • interviewRequired가 true일 때, status가 'APPROVED'가 아닌 경우에도 인터뷰 시간 필드에 하이픈(-)을 표시하도록 조건부 렌더링 로직을 변경하여 UI 일관성을 확보했습니다.
Activity
  • 제공된 컨텍스트에는 PR 활동에 대한 정보가 없습니다.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

안녕하세요. grid-template-columns 문자열을 만드는 과정에서 공백이 누락되어 발생한 레이아웃 문제를 수정한 점 확인했습니다. 올바른 수정입니다. 더불어 ApplicantListItem.tsx 파일에서 interviewRequired일 때 항상 면접 시간 열이 표시되도록 변경하여 그리드 레이아웃을 일관성 있게 유지한 점도 좋은 변경으로 보입니다.

리뷰에서는 styled.ts 파일에서 문자열 연결 대신 템플릿 리터럴을 사용하여 코드를 더 간결하고 안전하게 개선하는 방법을 제안했습니다. 이렇게 하면 이번에 수정하신 것과 같은 공백 관련 실수를 예방하는 데 도움이 될 것입니다. 감사합니다.

Comment on lines 9 to 10
grid-template-columns: ${({ hasInterview }) =>
'1fr 1fr 1fr 1.2fr 1.5fr 1fr' + (hasInterview ? '1.2fr' : '')};
'1fr 1fr 1fr 1.2fr 1.5fr 1fr' + (hasInterview ? ' 1.2fr' : '')};
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

안녕하세요. grid-template-columns 속성을 동적으로 구성할 때, 문자열 연결(+) 대신 템플릿 리터럴(template literals)을 사용하면 코드가 더 간결하고 가독성이 좋아집니다. 특히 공백 관리가 쉬워져 이번 PR에서 수정하신 것과 같은 실수를 방지하는 데 도움이 됩니다.

Suggested change
grid-template-columns: ${({ hasInterview }) =>
'1fr 1fr 1fr 1.2fr 1.5fr 1fr' + (hasInterview ? '1.2fr' : '')};
'1fr 1fr 1fr 1.2fr 1.5fr 1fr' + (hasInterview ? ' 1.2fr' : '')};
grid-template-columns: ${({ hasInterview }) =>
`1fr 1fr 1fr 1.2fr 1.5fr 1fr${hasInterview ? ' 1.2fr' : ''}`};

Comment on lines 20 to 21
grid-template-columns: ${({ hasInterview }) =>
'1fr 1fr 1fr 1.5fr 1fr' + (hasInterview ? '1.2fr' : '')};
'1fr 1fr 1fr 1.5fr 1fr' + (hasInterview ? ' 1.2fr' : '')};
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

이 부분도 템플릿 리터럴을 사용하면 더 명확하게 코드를 작성할 수 있습니다.

Suggested change
grid-template-columns: ${({ hasInterview }) =>
'1fr 1fr 1fr 1.5fr 1fr' + (hasInterview ? '1.2fr' : '')};
'1fr 1fr 1fr 1.5fr 1fr' + (hasInterview ? ' 1.2fr' : '')};
grid-template-columns: ${({ hasInterview }) =>
`1fr 1fr 1fr 1.5fr 1fr${hasInterview ? ' 1.2fr' : ''}`};

Comment on lines 29 to 30
grid-template-columns: ${({ hasInterview }) =>
'1fr 1fr 1.5fr 1fr' + (hasInterview ? '1.2fr' : '')}
'1fr 1fr 1.5fr 1fr' + (hasInterview ? ' 1.2fr' : '')};
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

여기에도 템플릿 리터럴을 적용하여 일관성을 유지하고 가독성을 높이는 것을 추천합니다.

Suggested change
grid-template-columns: ${({ hasInterview }) =>
'1fr 1fr 1.5fr 1fr' + (hasInterview ? '1.2fr' : '')}
'1fr 1fr 1.5fr 1fr' + (hasInterview ? ' 1.2fr' : '')};
grid-template-columns: ${({ hasInterview }) =>
`1fr 1fr 1.5fr 1fr${hasInterview ? ' 1.2fr' : ''}`};

Comment on lines 38 to 39
grid-template-columns: ${({ hasInterview }) =>
'1fr 1.5fr 1fr' + (hasInterview ? ' 1.2fr' : '')};
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

마지막으로 이 부분도 템플릿 리터럴로 변경하면 좋겠습니다. 이렇게 하면 프로젝트 전체의 코드 스타일이 일관되어 유지보수에 도움이 됩니다.

Suggested change
grid-template-columns: ${({ hasInterview }) =>
'1fr 1.5fr 1fr' + (hasInterview ? ' 1.2fr' : '')};
grid-template-columns: ${({ hasInterview }) =>
`1fr 1.5fr 1fr${hasInterview ? ' 1.2fr' : ''}`};

@kanghaeun kanghaeun merged commit b906aa1 into develop Feb 8, 2026
5 checks passed
@kanghaeun kanghaeun deleted the feat/dashboard-column-update branch February 8, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug 버그 이슈

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant