6주차 Assignment - 박지균 #12
Open
Conversation
This file contains 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
staccato20
reviewed
Nov 9, 2023
Comment on lines
+11
to
+13
| <h3><a href="{% url 'detail' blogs.id %}">{{ blogs.title }}</a></h3> | ||
| <p>{{ blogs.pub_date }}</p> | ||
| <p>{{ blogs.body }}</p> |
There was a problem hiding this comment.
가이드라인 미준수
read 페이지에서도 북마크 아이콘과 함께 북마크 기능을 구현해주셔야 합니다.
staccato20
reviewed
Nov 11, 2023
There was a problem hiding this comment.
가이드라인 미준수
현재 Font Awesome 라이브러리에서 가져온 모든 아이콘들이 detail 페이지를 제외하고는 화면 상에 보여지지 않고 있습니다. 강의자료를 다시 확인하여 아이콘들을 서버에서 불러올 수 있도록 수정해주세요.
힌트😉 : 강의자료만 보고 한다면 아이콘들이 전부 보여지지는 않을 것입니다.(그럼에도 우선 일부라도 보여지게 해야 어떤 부분을 고칠 수 있을지 갈피를 잡을 수 있을테니 강의자료를 먼저 보는 것이 좋을 것입니다.) 그 이유는 현재 Fomt Awesome 라이브러리의 여러 버전을 혼용하여 사용하였기 때문인데요, 버전을 하나로 통일하거나 혹은, 모든 버전을 아우를 수 있는 새로운 방식을 도입하여야 합니다!
Comment on lines
+14
to
+23
| <form action="{% url 'bookmark' blogs.id %}" method='POST'> | ||
| {% csrf_token %} | ||
| <button style="border: none"> | ||
| {% if user in blogs.bookmark_user.all %} | ||
| <i class ="fas fa-solid fa-bookmark" style="color: black; font-size:17px;"></i> | ||
| {% else %} | ||
| <i class ="far fa-regular fa-bookmark" style="color: black; font-size:17px;"></i> | ||
| {% endif %} | ||
| </button> | ||
| </form> |
There was a problem hiding this comment.
피드백
가이드라인은 통과했지만 북마크를 누르면 북마크만 눌리는 게 아니라 무조건 해당 글의 detail로 들어가지는 점이 아쉽습니다. 추후 본 과제를 더 발전시켜본다면 이 부분을 수정해보는 것도 좋을 것 같습니다 :)
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.
Description
Reference
Question