Skip to content

Conversation

@2-pi-r
Copy link
Contributor

@2-pi-r 2-pi-r commented Nov 15, 2022

과제3 (11/15수업 때 공지, 11/21 마감인 과제.)

한 것

  • 공통부분 처리했고,

  • 네비게이션 바 수정했고,

  • CSS 하면서 갈아엎은 html 파일에 대해서, 전체적으로 리뷰 달았어.

내가 그걸 이해해야 공통 부분을 처리할 수 있어서, 쭉 읽어봤거든.
그런데 내가 만든 게 아니라 함부로 수정할 수가 없어서, 리뷰로 남겼어. 담당자들 확인해줘!

안 한 것

  • 인터렉션 수정은 안 했음. 나중에 하자!

원래는 네비게이션 바 수정하고 그에 따라서 인터렉션 수정하기로 했었잖아.
그런데 막상 하려고 보니까 url에 get방식으로 인자 넣는 것도 배우고 나서 해야겠더라고.

인터렉션 수정할 내용 정리해둘게.

  • 맛집 리스트 페이지 --> 상세페이지 (사진 클릭)
  • 리뷰 조회 페이지 --> 식당 세부 화면 (버튼), 리뷰 등록 (버튼)
  • 메뉴 조회 페이지 --> 대표메뉴 등록 (버튼)
  • 상세페이지 --> 대표메뉴 조회 (버튼), 리뷰 조회 (버튼)

2-pi-r added 3 commits November 15, 2022 21:33
@application.route("/")에서 연결 페이지 수정
코드 정렬했더니 엄청 많이 수정한 것처럼 보이는데,
거의 공통부분 처리만 했어.
- 공통부분 처리 --> 다음 부분들을 {% block ~~~ %} 으로 묶음.
    head_title : <!--head 태그 안에 있는 title 태그 내용-->
    section : <!--각 페이지 내용-->
    co_side : <!--이 위치에 내용 비어있는 div태그가 있는 페이지가 있어서 추가함.-->

- 네비게이션 바 수정 (아래 항목 삭제)
    메뉴등록 ("/menuRegister")
    리뷰등록("/menuRegister")
    리뷰조회("/reviewView")
    메뉴 조회("/menuView")
    상세페이지("/detail")

- 헤더에 링크 추가 --> 홈(href="/")으로 이동하도록.
application = Flask(__name__)


@application.route("/")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

start3.html이 아무 url에서도 연결되지 않더라고.
그래서 아예 사용되질 않는데,
이거 start3.html 파일 아는 사람이 처리해줬으면 좋겠어!

혹시 빼기로 한 거면 아예 파일을 0temp폴더로 옮겨주면 좋을 것 같아.

일단 홈에는 list.html만 연결해놨어.

{% endblock head_title %}


<div class="리스트"></div>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

CSS파일 만든 사람들!
이거 빼도 돼?
왜 있는지 잘 모르겠는데, 혹시 넣은 의도가 있는 걸까 해서 물어봐.
그리고 만약에 뺀다면, whole.css에서 .리스트{ } 부분도 지워야 할 것 같아.

Copy link
Contributor

@yooksj13 yooksj13 Nov 15, 2022

Choose a reason for hiding this comment

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

이건 중간에 만들었다가 다른걸로 대체했던 클래스였던것 같아서 없애도 될거같아

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정 완료!

이대 주변 맛집 소식을<br>
봐주세요~</p>
</div>
<div class="맛집 상세페이지"></div>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

CSS파일 만든 사람들!
이것도 빼도 돼?
얘도 왜 있는지 잘 모르겠는데, 혹시 넣은 의도가 있는 걸까 해서 물어봐.
이거는 whole.css에서 지정한 부분도 없더라고.

Copy link
Contributor

Choose a reason for hiding this comment

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

이런건 원래 있었던걸로 기억해 우리가 넣은 클래스들은 다 css에서 하려고 넣은 것들이라 css에 없으면 아마 원래 있었던 것들일거야. 그니까 빼도 아무 상관 없어용

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정 완료!

</nav>


<!--사이드바-->
Copy link
Contributor Author

Choose a reason for hiding this comment

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

CSS 파일 만든 사람들!
여기 aside태그를 사용하지 않은 이유가 있을까?
이유가 있어서 안 쓴 거라면, div태그로라도 사이드바를 한 덩어리로 묶는 건 어때?
지금은 card 여러 개가 따로 있잖아.

Copy link
Contributor

Choose a reason for hiding this comment

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

사이드바는 co side 안에 묶여져 있고 사이드바 안에 있는 것들은 네모 두개로 따로 보이게하려구 일부러 card를 각각 쓴거야. 인기차트랑 followme가 주제가 다른데 같은 카드안에 묶여서 보이면 좀 이상할거같아서 그렇게 했엉

Copy link
Contributor Author

Choose a reason for hiding this comment

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

응응 그럼 aside태그 추가로 넣지 않고 그대로 둘게!

<!--각 페이지 section-->
<section>
<div class="co mid">
{% block section %}
Copy link
Contributor Author

@2-pi-r 2-pi-r Nov 15, 2022

Choose a reason for hiding this comment

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

(이 리뷰는 뭘 물어보는 건 아니고, 그냥 그렇다고 db용으로 남겨두는 거야.)

이 안에서 페이지 위에 오는 제목도 묶고 싶었는데, 문제가 생기는 것 같아서 뺐어.
block 안에 block을 넣으니까 작동이 원하는 대로 안 되나봐.
게다가 html 보면 어떤 페이지들에선 제목 부분 코드가 달라서, 묶으면 처리가 힘들더라고.

2-pi-r and others added 7 commits November 15, 2022 22:31
피드백 반영해서 불필요한 부분 삭제함
HW01, HW02, HW03 at one time
eliminate menuRegister in navigation bar
eliminate menuRegister from navigation bar
eliminate menuRegister from navigation bar
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.

5 participants