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: replace TextEditor to Markdown #1231

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

jinhyeonkwon
Copy link
Contributor

요약 *

It closes #1230

구현한 것

  1. 안건 수정 시, 안건 내용을 입력하는 부분을 Markdown component로 교체했습니다.
  2. 이에 대응하여, NewAgendaFrame, AgendaBlock, AgendaEditor의 작동 로직을 변경했니다.
    기존에 수정하다가 접거나 순서를 바도 바로 저장되어버리는 문제가 있었는데, 취소 버튼을 집어넣으면서, 저장을 누르기 전까지는 가장 하위 컴포넌트인 AgendaEditor의 내부 state로 변경 사항을 관리하다가 저장을 해야 반영되도록 변경했습니다.
  3. Markdown component에 대한 수정도 있습니다. @ChaeyeonAhn 님께서 봐주시면 좋을 것 같습니다! 초깃값 (수정 상황이므로 기존의 값이 들어 있게) 을 전달하기 위한 prop(initialValue)과 값 변경 시 상위 컴포넌트가 이를 감지하여 내용을 받아올 수 있도록 하는 콜백 함수(onChange)를 추가했습니다.

특이사항

  1. 아래와 같이 721-960px 범위에서 Markdown component의 버튼이 튀어나갑니다!
    image
  2. 수정 중에 옮기면 이제 수정 전의 내용이 복구됩니다. 그러나 수정 사항이 다 날아가는 것도 유쾌한 경험은 아닐 것 같습니다. 수정 중에는 문제가 발생하지 않도록 안건 순서 변경이나 접기를 막아도 좋을 것 같습니다.
  3. 수정 완료 후, html 형태로 저장된 내용이 그대로 보여서 불편할 것 같습니다. 추후 작업을 통해, AgendaViewer에서도 html 형태를 대응하면 좋을 것 같습니다.
  4. Markdown에서 마지막 키보드 입력이 사라질 때가 있습니다! Markdown component의 문제인지, 그것을 사용하는 AgendaEditor나 다른 component의 문제인지는 아직 잘 모르겠습니다.

스크린샷

image
image

이후 Task *

  • 없음

@jinhyeonkwon jinhyeonkwon linked an issue Nov 26, 2024 that may be closed by this pull request
2 tasks
@ChaeyeonAhn
Copy link
Contributor

저는 마크다운 index.ts 고치신 것만 봤는데 좋아보여요! 👍

Comment on lines 23 to +29
// TODO: Impl TextArea
const TextEditor = styled.input.withConfig({
shouldForwardProp: prop => isPropValid(prop),
})`
border: 1px solid ${({ theme }) => theme.colors.PRIMARY};
padding: 32px;
`;
// const TextEditor = styled.input.withConfig({
// shouldForwardProp: prop => isPropValid(prop),
// })`
// border: 1px solid ${({ theme }) => theme.colors.PRIMARY};
// padding: 32px;
// `;
Copy link
Contributor

Choose a reason for hiding this comment

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

요거 지워도 될듯!

@babycroc
Copy link
Contributor

마지막 키보드 입력 사라짐 -> 이거 설마 윈도우 이슈일 수도 있으려나....????? 요거 관련해서는 좀 더 검색해볼게 들어본 적은 있어,,,,

특이사항에 써준 2, 3번 마무리 작업만 부탁합니당!

@babycroc
Copy link
Contributor

마지막 키보드 입력 사라짐 -> 이거 설마 윈도우 이슈일 수도 있으려나....????? 요거 관련해서는 좀 더 검색해볼게 들어본 적은 있어,,,,

특이사항에 써준 2, 3번 마무리 작업만 부탁합니당!

오 마자 비서에서 비슷?한 걸 봤던 것 같은데 그건 요거야 sparcs-kaist/biseo#195

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.

add markdown editor to agenda edit container
3 participants