Skip to content

Comments

fix: #805 템플릿 질문 리스트 편집 후 완료버튼 클릭 시 수정 완료 토스트 제공#807

Merged
prgmr99 merged 3 commits intodevelopfrom
805-템플릿-질문-리스트-질문-리스트-편집-후-완료버튼-클릭-시-수정-완료-토스트-제공
Feb 18, 2026

Hidden character warning

The head ref may contain hidden characters: "805-\ud15c\ud50c\ub9bf-\uc9c8\ubb38-\ub9ac\uc2a4\ud2b8-\uc9c8\ubb38-\ub9ac\uc2a4\ud2b8-\ud3b8\uc9d1-\ud6c4-\uc644\ub8cc\ubc84\ud2bc-\ud074\ub9ad-\uc2dc-\uc218\uc815-\uc644\ub8cc-\ud1a0\uc2a4\ud2b8-\uc81c\uacf5"
Merged

fix: #805 템플릿 질문 리스트 편집 후 완료버튼 클릭 시 수정 완료 토스트 제공#807
prgmr99 merged 3 commits intodevelopfrom
805-템플릿-질문-리스트-질문-리스트-편집-후-완료버튼-클릭-시-수정-완료-토스트-제공

Conversation

@prgmr99
Copy link
Member

@prgmr99 prgmr99 commented Feb 11, 2026

템플릿 질문 리스트 편집 후 완료버튼 클릭 시 수정 완료 토스트 제공


🏄🏼‍♂️‍ Summary (요약)

  • 수정 완료 시, 수정 완료 토스트 제공하도록 추가했습니다.

🫨 Describe your Change (변경사항)

  • 질문 수정 후 토스트 추가
  • 아이콘 padding 제거

🧐 Issue number and link (참고)

📚 Reference (참조)

2026-02-11.3.53.42.mov

++영상에서는 삭제버튼 클릭 시, 리스트의 높이가 변하는데 이 점도 함께 수정했습니다.

@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • Not Rabbit

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 805-템플릿-질문-리스트-질문-리스트-편집-후-완료버튼-클릭-시-수정-완료-토스트-제공

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

display: flex;
align-items: center;
padding: 0.4rem;
/* padding: 0.4rem; */
Copy link
Member

Choose a reason for hiding this comment

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

혹시 몰라서 남겨두신 것 같은데 안쓰는 패딩이면 그냥 제거해도 될 듯 해요!

Copy link
Member Author

Choose a reason for hiding this comment

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

넵!! 제거해놓겠습니다!

Copy link
Member

@klmhyeonwoo klmhyeonwoo left a comment

Choose a reason for hiding this comment

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

LGTM!

@prgmr99 prgmr99 force-pushed the 805-템플릿-질문-리스트-질문-리스트-편집-후-완료버튼-클릭-시-수정-완료-토스트-제공 branch from a5c8ac6 to a0b82dc Compare February 18, 2026 11:20
@prgmr99 prgmr99 merged commit c880242 into develop Feb 18, 2026
2 checks passed
@prgmr99 prgmr99 deleted the 805-템플릿-질문-리스트-질문-리스트-편집-후-완료버튼-클릭-시-수정-완료-토스트-제공 branch February 18, 2026 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants