-
Notifications
You must be signed in to change notification settings - Fork 1
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
base: dev
Are you sure you want to change the base?
feat: replace TextEditor to Markdown #1231
Conversation
저는 마크다운 index.ts 고치신 것만 봤는데 좋아보여요! 👍 |
// 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; | ||
// `; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요거 지워도 될듯!
마지막 키보드 입력 사라짐 -> 이거 설마 윈도우 이슈일 수도 있으려나....????? 요거 관련해서는 좀 더 검색해볼게 들어본 적은 있어,,,, 특이사항에 써준 2, 3번 마무리 작업만 부탁합니당! |
오 마자 비서에서 비슷?한 걸 봤던 것 같은데 그건 요거야 sparcs-kaist/biseo#195 |
요약 *
It closes #1230
구현한 것
Markdown
component로 교체했습니다.NewAgendaFrame
,AgendaBlock
,AgendaEditor
의 작동 로직을 변경했니다.기존에 수정하다가 접거나 순서를 바도 바로 저장되어버리는 문제가 있었는데, 취소 버튼을 집어넣으면서, 저장을 누르기 전까지는 가장 하위 컴포넌트인
AgendaEditor
의 내부 state로 변경 사항을 관리하다가 저장을 해야 반영되도록 변경했습니다.Markdown
component에 대한 수정도 있습니다. @ChaeyeonAhn 님께서 봐주시면 좋을 것 같습니다! 초깃값 (수정 상황이므로 기존의 값이 들어 있게) 을 전달하기 위한 prop(initialValue
)과 값 변경 시 상위 컴포넌트가 이를 감지하여 내용을 받아올 수 있도록 하는 콜백 함수(onChange
)를 추가했습니다.특이사항
Markdown
component의 버튼이 튀어나갑니다!AgendaViewer
에서도 html 형태를 대응하면 좋을 것 같습니다.Markdown
component의 문제인지, 그것을 사용하는AgendaEditor
나 다른 component의 문제인지는 아직 잘 모르겠습니다.스크린샷
이후 Task *