Skip to content
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

Frontend #29

Open
wants to merge 70 commits into
base: master
Choose a base branch
from
Open

Frontend #29

wants to merge 70 commits into from

Conversation

Jinu-Lee
Copy link
Member

  • Header
    딱히 문제 없어보임

  • Footer

    1. 회장/부회장이 매번 바뀔텐데 그때마다 코드 수정을 해야하는건지?
    2. Github / 백준 바로가기 => NPC 깃헙 / 백준 그룹을 링크하는건지?
  • 인트로 ('/')
    딱히 문제 없어보임

  • 메인 ('/main')

    1. 맨 아래 '지원하기' 누르면 뭐가 나오는건지? (회원가입?)
    2. 비율이 디자인과는 조금 다름
  • 로그인 / 회원가입 / 비밀번호 찾기
    딱히 문제 없어보임

  • 멤버 ('/member')

    1. Profile 중에 Github ID도 있는데 이는 왜 없는지?
    2. 신규 가입 창에서 디자인 상에는 학번?이 있던데 DB에는 학번이 없음
      => 신규 가입자의 어떤 정보를 출력할지 (현재는 이름, 이메일, 학과)
  • 일정 ('/calendar')
    딱히 문제 없어보임

  • 스터디 ('/study')
    성현님이 작성한 코드가 잘 안 돌아가서 새로 작성함 -> 작업 중...

  • 설정 ('/user/settings')

    1. 한글이랑 영어가 섞여있어서 한글로 통일
    2. 기본 이미지 다른거로?
  1. 지금 layout 중에 화면 비율로 되어있지 않고 고정 픽셀로 되어있는 데가
    많은데 제 모니터 해상도가 높아서 그런지 전체 화면으로 봤을 때
    화면이 잘리거나 그러진 않았습니다.
    혹시 잘리는 부분 있으면 comment해주세요

  2. 급하게 하다보니 임의로 디자인/수정한 부분이 조금 있습니다.
    디자인 중에 수정할 부분 comment해주세요

@threedalpeng
Copy link
Member

threedalpeng commented Feb 28, 2021

전체적으로 마우스 포인터가 상황과 역할에 따라 적절하게 변경되도록 수정해주면 좋겠습니다.

  • 맨 처음 위로 슬라이드 되는 부분
  • 스터디 페이지의 버튼들
  • 이외 확인하지 못한 여러 부분들

@threedalpeng
Copy link
Member

threedalpeng commented Feb 28, 2021

추가적으로 스터디의 편집창에서 "제목을 입력하세요..." 등을 placeholder로 지정해놓으면 좋을 것 같습니다.

@Jinu-Lee
Copy link
Member Author

Jinu-Lee commented Mar 1, 2021

@threedalpeng

찾아보니 Link 이외의 element는 cursor: pointer로 쓰지 말라는 의견이 많네요
일관성 있게 수정했기는 한데 혹시 튀는 부분 있으면 comment 달아주세요

https://ux.stackexchange.com/questions/105024/why-dont-button-html-elements-have-a-css-cursor-pointer-by-default

@dotoleeoak
Copy link
Member

Footer

회장/부회장이 매번 바뀔텐데 그때마다 코드 수정을 해야하는건지?
Github / 백준 바로가기 => NPC 깃헙 / 백준 그룹을 링크하는건지?

모두 Yes
회장/부회장 바꾸는건 서버에서 바로 코드를 수정해도 됨

메인 ('/main')

맨 아래 '지원하기' 누르면 뭐가 나오는건지? (회원가입?)

Google Form (평소에는 닫아둠)

멤버 ('/member')

Profile 중에 Github ID도 있는데 이는 왜 없는지?

디자인에서 누락된 듯. 추가해도 됨

신규 가입 창에서 디자인 상에는 학번?이 있던데 DB에는 학번이 없음
=> 신규 가입자의 어떤 정보를 출력할지 (현재는 이름, 이메일, 학과)

DB에 학번 추가하는게 좋아보임 (전체 학번말고 연도만)
신규 가입자는 전부 출력해도 될듯? (이름, 이메일, 학과, 학번) 큰 상관 없음

설정 ('/user/settings')

기본 이미지 다른거로?

보류

지금 layout 중에 화면 비율로 되어있지 않고 고정 픽셀로 되어있는 데가
많은데 제 모니터 해상도가 높아서 그런지 전체 화면으로 봤을 때
화면이 잘리거나 그러진 않았습니다.
혹시 잘리는 부분 있으면 comment해주세요

OK. 근데 지금 당장은 아니더라도 상대값(rem, em이나 %)으로 바꾸는게 좋을 듯

@threedalpeng
Copy link
Member

threedalpeng commented Mar 1, 2021

@Jinu-Lee 적어도 가시적으로 동작하는 버튼임을 알아볼 수 있는 조치는 있으면 (hover 시 스타일이 달라지거나..) 좋을 것 같지만... 당장 디자인을 처리하기 곤란하다면 알겠습니다.

추가) 아 일관성 있게 바꾸셨다고 하셨네요. 제 눈이 어두웠습니다...

@Jinu-Lee
Copy link
Member Author

Jinu-Lee commented Mar 1, 2021

@threedalpeng
음 일단 버튼은 누가 봐도 버튼처럼 보이게끔
(경계, 배경 색 등 설정) 수정해보겠습니다

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.

6 participants