swsnu / swppfall2022-team4

4 stars 1 forks source link

[FEAT] workout_sprint5 #94

Closed jihyungKo closed 1 year ago

jihyungKo commented 1 year ago
jihyungKo commented 1 year ago
jihyungKo commented 1 year ago
jihyungKo commented 1 year ago

@blackruby02 @KJYoung @seuha516 PR 활성화했습니다!

KJYoung commented 1 year ago

확인했습니다! 사소한 한가지만 부탁드리면 이미지에 hover했을 때 나오는 삭제 버튼이 Post에서 쓰던 삭제버튼처럼 중간에 나오는게 아니라 우상단에 나오는게 나을 것 같아요(Post에서 hover-삭제 버튼이 나오는 상황은 글을 작성/편집중일 때라 눌렀을 때 modal이 나오지는 않아서 상관없는데, 여기서는 삭제버튼을 누르면 삭제, 이미지를 누르면 modal이 나와야 해서 아무생각없이 중간을 눌러 자세히 보려다가 삭제 되는 상황이 많이 생길 것 같아요)

KJYoung commented 1 year ago

여담으로 drag-and-drop은 태그에서 제가 구현하려다가 여러줄의 리스트는 안돼서 포기했었는데, 여기서는 한 줄로만 리스트가 보여지다보니 적용하기가 좋은 예시였네요, 잘 되는 것 확인했습니다 수고하셨습니다!

jihyungKo commented 1 year ago

확인했습니다! 사소한 한가지만 부탁드리면 이미지에 hover했을 때 나오는 삭제 버튼이 Post에서 쓰던 삭제버튼처럼 중간에 나오는게 아니라 우상단에 나오는게 나을 것 같아요(Post에서 hover-삭제 버튼이 나오는 상황은 글을 작성/편집중일 때라 눌렀을 때 modal이 나오지는 않아서 상관없는데, 여기서는 삭제버튼을 누르면 삭제, 이미지를 누르면 modal이 나와야 해서 아무생각없이 중간을 눌러 자세히 보려다가 삭제 되는 상황이 많이 생길 것 같아요)

아 그럴 수 있겠네요!! 지금은 밖이라 저녁 늦은 시간에 수정하겠습니다😄

jihyungKo commented 1 year ago

@blackruby02 @KJYoung @seuha516 수정완료했습니다! travis 검사 끝나면 아무나 머지 부탁드립니다.

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

86.8% 86.8% Coverage
2.2% 2.2% Duplication