Fix(#272, #274, #276): 피드백에서 제보된 메모 관련 문제 해결 #277
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 메모지 1개만 선택했을 때 편집 패널이 나오도록 수정
- 백스페이스를 누르면 객체를 지우는 이벤트가 발생. 텍스트를 편집 중인가 여부를 포함하는 상태를 하나 만들어 이를 추적해 이벤트를 등록, 제거해 해결
✅ Deploy Preview for boarlog ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Jw705
approved these changes
Dec 12, 2023
| @@ -132,6 +147,7 @@ const Toolbar = () => { | |||
|
|
|||
| // 더미 텍스트박스를 생성합니다. | |||
| const dummyTextBox = new fabric.Textbox(textBox.text, { | |||
| fontFamily: "Pretendard Variable", | |||
LellowMellow
approved these changes
Dec 12, 2023
| @@ -27,8 +29,12 @@ const QuestionList = () => { | |||
| <h2 className="semibold-18 inline-block mt-1 p-4">질문 리스트</h2> | |||
| <div className="h-[36rem] px-4 overflow-y-auto"> | |||
| <ul ref={listRef}> | |||
| {questions.map((question, index) => ( | |||
| <li className={`p-4 h-fit mb-4 min-h-[6.25rem] ${MEMO_COLOR} relative`} key={index}> | |||
| {questions.map(({ content, questionId }, index) => ( | |||
Comment on lines
+97
to
+98
| const textBoxHeight = textBox.get("height") <= 145 ? 145 : textBox.get("height") + 20; | ||
| memoBackground.set({ height: textBoxHeight }); |
| const textBoxHeight = textBox.get("height") <= 145 ? 145 : textBox.get("height") + 20; | ||
| memoBackground.set({ height: textBoxHeight }); | ||
|
|
||
| // @ts-ignore |
There was a problem hiding this comment.
화나는 ts-ignore....... 진짜 ㅠㅠㅠ webRTC 관련해서 처리한 것처럼 저희도 직접 type을 지정...해...줄까요...?
| const canvas = useRecoilValue(canvasInstanceState); | ||
| const setVisibilityEditPanel = useSetRecoilState(stickyNoteEditPanelVisibilityState); | ||
| const setStickyNoteInstance = useSetRecoilState(stickyNoteInstance); | ||
| const questionContents = useRecoilValue(clickedQuestionContentsState); | ||
| const setDefaultQuestionContents = useResetRecoilState(clickedQuestionContentsState); | ||
| const setIsQuestionListOpen = useSetRecoilState(isQuestionListOpenState); | ||
| const questionSocket = useRecoilValue(instructorSocketRefState); | ||
| const roomId = new URLSearchParams(useLocation().search).get("roomid") || "999999"; |
There was a problem hiding this comment.
이것도 뭔가 999999 말고 나중에 무조건 값을 받아오게끔? 하는건 어떨까요? 없으면 잘못된 접근이라고 error로 넘겨버리는거죠.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
작업 개요
메모지 영어, 한글 폰트가 다른 문제 해결, 폰트 크기 제어시 메모지 배경 크기 조절 close #272
메모지와 다른 요소를 같이 선택할 경우 메모 편집 패널 표시 오류 close #274
메모지 입력 시 백스페이스 문제 해결 close #276
작업 사항
메모지 영어, 한글 폰트가 다른 문제 해결, 폰트 크기 제어시 메모지 배경 크기 조절
해당 조작시 배경이 되는 Rect 객체의 height를 늘려주는 것으로 해결했습니다.
메모지와 다른 요소를 같이 선택할 경우 메모 편집 패널 표시 오류
메모지 1개만 선택했을 때 편집 패널이 나오도록 수정
메모지 입력 시 백스페이스 문제 해결
백스페이스를 누르면 객체를 지우는 이벤트가 발생. 텍스트를 편집 중인가 여부를 포함하는 상태(isMemoEditingState)를 만들어 이를 추적해 이벤트를 등록, 제거해 해결
고민한 점들(필수 X)
스크린샷(필수 X)