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

[Refactor] UI 개선 - Code 길이 #323

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

dlsxjzld
Copy link
Contributor

@dlsxjzld dlsxjzld commented Apr 27, 2024

📋 Issue Number

close #

  • code 문제의 한 줄의 길이가 70을 넘어가면, 입력창을 넘어가는 문제가 있었습니다.
70자 넘어가면 input창에서 잘 안보임

💻 구현 내용

  • code 문제의 한 줄의 길이가 65를 넘어가면, 한 줄에 포함된 공백의 개수에 따라 문제를 적절하게 잘랐습니다.

📷 Screenshots

65자 넘어가면 입력란에서 많이 길어서 자름_1 65자 넘어가면 입력란에서 많이 길어서 자름_2

🤔 고민사항

  • 화면에 보이는 문제는 템플릿(공백, 줄넘김)이 적용되어 있는데 2가지 문제가 있습니다.
  1. 좌우 길이가 길면 화면에서 문제가 잘리고 스크롤이 생깁니다. (사진 속 문제의 2번째 줄 참고)

    • code 좌우 길이가 너무 길어서 잘림
  2. 상하 길이가 길면 트랙을 침범합니다.

    • code 상하 길이가 너무 길어서 track 위로 넘침
  • highlight.js 가 적용된 문제 위에 글자만 색상을 입히는 방법은 찾아내지 못했습니다.. 😂
  • 디자인을 바꾸게 된다면 트랙, 타수 그리고 정확도 계기판의 위치 조정도 필요할 것 같습니다!
  • 좋은 의견 있다면 말씀해주세요!!

@dlsxjzld dlsxjzld self-assigned this Apr 27, 2024
Copy link
Member

@yejinleee yejinleee left a comment

Choose a reason for hiding this comment

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

오 수식으로 줄넘김처리를 하셨군요 👏👏

여긴 디자인분들께도 한번 더 언급드려야할 부분이겠네요 8ㅅ8

코드에선 계기판이 없어지던지 더 줄어들던지,,, 그렇게 되면 한줄(가로길이)이 길어져도 상관없을 것 같거든요!

@@ -10,7 +10,7 @@ const CodeContainer = ({ codeItem }: CodeContainerProps) => {
return (
<>
<div className='w-[60rem] select-none grow'>
<Highlight className='javascript'>{codeItem}</Highlight>
<Highlight className={'javascript' || 'java'}>{codeItem}</Highlight>
Copy link
Member

@yejinleee yejinleee Apr 29, 2024

Choose a reason for hiding this comment

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

오옷 이렇게 하면 알아서 js거나 java로 하이라이팅이 되나요!!? 따로 인자를 넘긴다거나 하는 작업 없이 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants