Skip to content

Conversation

@skinmaker1345
Copy link
Member

  • 최상단 헤더에 봇 / 서버를 표기하여 UX를 개선합니다.
image image

Copy link
Member

@wonderlandpark wonderlandpark left a comment

Choose a reason for hiding this comment

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

possible undefined returning for type all

<Container className='pb-16 pt-24 md:pb-20' ignoreColor>
<h1 className='hidden text-left text-3xl font-bold md:block'>
{header && `${header} - `}한국 디스코드 리스트
{header && `${header} - `} {{ bots: '봇 | ', servers: '서버 | ' }[type]}한국 디스코드
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
{header && `${header} - `} {{ bots: '봇 | ', servers: '서버 | ' }[type]}한국 디스코드
{header && `${header} - `} {{ bots: '봇 | ', servers: '서버 | ' }[type] || ''}한국 디스코드

</span>
)}
한국 디스코드 리스트
{{ bots: '봇 | ', servers: '서버 | ' }[type]}한국 디스코드 리스트
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
{{ bots: '봇 | ', servers: '서버 | ' }[type]}한국 디스코드 리스트
{{ bots: '봇 | ', servers: '서버 | ' }[type] || ''}한국 디스코드 리스트

@skinmaker1345 skinmaker1345 merged commit 8ab5e43 into master Apr 11, 2025
3 checks passed
@skinmaker1345 skinmaker1345 deleted the feat/show-header-with-type branch April 11, 2025 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants