Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feat] 투표 옵션에 이미지, 코드블럭 추가 #68

Merged
merged 18 commits into from
Feb 1, 2023

Conversation

hwookim
Copy link
Collaborator

@hwookim hwookim commented Jan 28, 2023

close #53

💡 개요

스크린샷 2023-01-28 오후 1 58 31
스크린샷 2023-01-28 오후 1 58 45

📝 작업 내용

  • SelectOption에 이미지, 코드블럭 기능 추가
  • CodeEditor 구현 <- 추후 생성 페이지에서도 이용할 예정

‼️ 주의 사항

  • 코드 블럭 표시 방식에 대해서는 논의 중인 내용이 있어서 이후 추가 작업이 필요할 것 같습니다!
  • next에서 사용 가능하도록 설정 변경한 부분이 있음 참고
  • 전체 document에 다크모드 활성화함

🔗 참고자료

- image mock data 추가
- 버튼으로 태그 변경
…d-plus

# Conflicts:
#	__mocks__/data/voteOption.ts
#	src/components/common/TopicCard/SelectOption/SelectOption.tsx
#	src/components/common/TopicCard/TopicCard.stories.tsx
#	src/components/common/TopicCard/TopicCard.tsx
#	src/components/topic/TopicDetailMain/TopicDetailMain.tsx
@hwookim hwookim added the ✨ feature 기능 개발 label Jan 28, 2023
@hwookim hwookim self-assigned this Jan 28, 2023
@hwookim hwookim changed the title Feat/053 topic card plus [feat] 투표 옵션에 이미지, 코드블럭 추가 Jan 28, 2023
Copy link
Collaborator

@KIMSEUNGGYU KIMSEUNGGYU left a comment

Choose a reason for hiding this comment

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

오오 점점 완성된 가는거 보니깐 좋네요!!
이번 UI 하면서 재미있었을거 같아요!!

__mocks__/data/voteOption.ts Show resolved Hide resolved
import theme from '@src/styles/theme';

export const CodeEditor = styled(TextAreaCodeEditor)`
${theme.font.code}
Copy link
Collaborator

Choose a reason for hiding this comment

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

스타일이 제대로 적용된건가요?? 속성이 정의가 되지 않고 값만 들어가 있는 거 같아요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

확인해보니 적용되고 있는 것 같은데 어떤 문제가 있을까요?
image

Copy link
Collaborator

Choose a reason for hiding this comment

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

image

  1. 스토리북에서 확인해보니 기본 폰트 color 가 어두운거 같은데 정상 동작하시나요??
    저희 배포된 스토리북(chromatic 및 local)으로 확인해도 해당 부분 컬러가 어둔운데 확인 부탁드립니다!
  2. codeEditor 부분이 viewer 하고 edit 모두 사용하는지 궁금합니다.
    만약 viewer 로만 사용한다면 네이밍을 CodeViwer가 더 낫지 않을까 싶습니다!


export interface CodeEditorProps extends TextareaCodeEditorProps {
value: string;
language: Languages;
Copy link
Collaborator

Choose a reason for hiding this comment

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

코드 에디터에 language 는 어떻게 사용하는건가요?
아직 제대로 적용이 안된건가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

정상 적용되고 있는데 storybook에 제대로 된 인자를 안넣어서 혼란을 드린 것 같네요.
관련 storybook 업데이트 했습니다!

</S.Info>
<S.Container>
<S.SelectButton $result={result} $selected={selected} onClick={handleClick}>
{result && <S.ProgressBar className="progress" $rate={rate} $selected={selected} />}
Copy link
Collaborator

Choose a reason for hiding this comment

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

해당 클래스컴포넌트에 progress 클래스 가 있는데 해당 부분 style 이 없는거 같은데 혹시 넣으신 의도가 궁금해요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

SelectButton 내에서 hover.progressbackground-color를 조정하는 로직이 있습니다!

@@ -7,7 +7,7 @@ import VoteOption from '@src/types/VoteOption';

import Icon from '../Icon';
import SelectOption from './SelectOption';
import * as S from './TopicCard.style';
import * as S from './TopicCard.styles';
Copy link
Collaborator

Choose a reason for hiding this comment

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

ㅋㅋㅋ 굿굿굿!! 👍👍👍😁

Copy link
Collaborator Author

@hwookim hwookim left a comment

Choose a reason for hiding this comment

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

맞아요 만들면서 너무 재밌었던 것 같아요 ㅋㅋㅋ

리뷰에 답변 남기면서 몇가지는 반영했습니다!
확인 한 번 부탁드려요 👍🏼

import theme from '@src/styles/theme';

export const CodeEditor = styled(TextAreaCodeEditor)`
${theme.font.code}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

확인해보니 적용되고 있는 것 같은데 어떤 문제가 있을까요?
image


export interface CodeEditorProps extends TextareaCodeEditorProps {
value: string;
language: Languages;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

정상 적용되고 있는데 storybook에 제대로 된 인자를 안넣어서 혼란을 드린 것 같네요.
관련 storybook 업데이트 했습니다!

</S.Info>
<S.Container>
<S.SelectButton $result={result} $selected={selected} onClick={handleClick}>
{result && <S.ProgressBar className="progress" $rate={rate} $selected={selected} />}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

SelectButton 내에서 hover.progressbackground-color를 조정하는 로직이 있습니다!

Copy link
Collaborator

@KIMSEUNGGYU KIMSEUNGGYU left a comment

Choose a reason for hiding this comment

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

CodeEditor 부분만 확인 부탁드립니다!
확인하시고 현우님 판단하에 처리하시면 괜찮을거 같아요!
나머지는 완벽합니다!! 👍👍👍

import theme from '@src/styles/theme';

export const CodeEditor = styled(TextAreaCodeEditor)`
${theme.font.code}
Copy link
Collaborator

Choose a reason for hiding this comment

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

image

  1. 스토리북에서 확인해보니 기본 폰트 color 가 어두운거 같은데 정상 동작하시나요??
    저희 배포된 스토리북(chromatic 및 local)으로 확인해도 해당 부분 컬러가 어둔운데 확인 부탁드립니다!
  2. codeEditor 부분이 viewer 하고 edit 모두 사용하는지 궁금합니다.
    만약 viewer 로만 사용한다면 네이밍을 CodeViwer가 더 낫지 않을까 싶습니다!

@hwookim
Copy link
Collaborator Author

hwookim commented Jan 30, 2023

@KIMSEUNGGYU
image
저는 정상 동작하는 걸로 보이는데,,, 어떤 문제일까요 😭
제 쪽에서 구현이 안되어서 가능하시다면 승규님께 확인 부탁드려도 괜찮을까요 ㅠㅠ

에디터에도 같이 쓸 생각으로 만든 거긴 합니다!
이 부분은 혜진님과도 논의가 한 번 더 필요할 것 같긴 하지만요.

Copy link
Collaborator

@KIMSEUNGGYU KIMSEUNGGYU left a comment

Choose a reason for hiding this comment

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

현우님 CodeEditor 스타일 적용확인했습니다.
근데 하나 더 이슈가 보입니다 ㅠㅠ

image
드래그 했을때 작은 글씨로 덮어쓰이는 거 같은데! 한 번 확인 부탁드립니다!

@hwookim
Copy link
Collaborator Author

hwookim commented Feb 1, 2023

후,,,, 되게 스타일이 이상하게 적용되고 있었네요...
에디터에서 textarea랑 실제 보여지는 text랑 따로 관리하다보니 폰트가 어그러져 있었습니다 ㅠㅠ
좀 더럽긴 하지만

export const CodeEditorWrapper = styled.div`
  ${theme.font.code}

  .w-tc-editor, * {
    ${theme.font.code}
  }
`;

이런 식으로 하위에 적용하니 되긴 하네요..!

Copy link
Collaborator

@KIMSEUNGGYU KIMSEUNGGYU left a comment

Choose a reason for hiding this comment

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

현우님 완벽합니다!!
정말 수고하셨어요~~

여태 일하면서 작업하시다니 대단합니다!! 👍👍👍

@hwookim hwookim merged commit 5d0a9bf into develop Feb 1, 2023
@hwookim hwookim deleted the feat/053-topic-card-plus branch February 1, 2023 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] Topic 카드에 이미지/코드 블럭 추가
2 participants