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

Create Header components #67

Closed
9 of 12 tasks
sosoYim opened this issue Jan 17, 2022 · 2 comments · Fixed by #114 or #129
Closed
9 of 12 tasks

Create Header components #67

sosoYim opened this issue Jan 17, 2022 · 2 comments · Fixed by #114 or #129
Assignees
Milestone

Comments

@sosoYim
Copy link
Contributor

sosoYim commented Jan 17, 2022

Todo

Assignee

ETC

  • Opinions & Problems
  • 컴포넌트 z-index 덮어지지 않게 조심하기 (Giphy 사이트에선 창이 작아질 때 검색바 아래로 드롭다운이 펼쳐짐)
@sosoYim sosoYim added this to the Sprint3 milestone Jan 17, 2022
@sosoYim sosoYim self-assigned this Jan 17, 2022
@sosoYim
Copy link
Contributor Author

sosoYim commented Jan 18, 2022

aria-expanded MDN

사용자 경험 관련 UX UI 고민

  • 국내에서 자주 볼 수 있는 잘못된 사용자 경험 : 컨텐츠로 가기 위해 모든 탭을 탐색해야하는 것.

  • 해외의 선례들은 운영체제 경험을 따라가는 추세다.

    • 탭 뿐만이 아니라 방향키를 사용할 수 있도록 하면 좋다.
    • aria-expanded: false? : 탐색할 것인지 아닌지 사용자에게 선택하게 해야 한다.
  • 바로 메뉴로 갈 수 있는 단축키 내지 버튼을 줘보는 건 어떨까? 한계를 두지말고 생각해보자.

    • 낯선 단축키? => 가이드를 제공. 이 후 가이드를 다시 볼지에 대한 정보를 캐싱하여 선택할 수 있게 하는 쪽으로 보완할 수도 있다.
  • wai-aria landmark role을 사용하면 각 영역들을 보조기기로 탐색할 수 있다.

  • 마우스 오버가 됐을 때 보이는 것은 마우스에 제한 적이다. 키보드 접근도 고려해야 함.

@sosoYim
Copy link
Contributor Author

sosoYim commented Jan 18, 2022

1040 을 기준으로 잡고 이상은 고정, 미만으로 내려가면 모바일형 네비로 전환하자.

@jkpark104 jkpark104 modified the milestones: Sprint3, Sprint4 Jan 23, 2022
@congaweb congaweb modified the milestones: Sprint3, Sprint4 Jan 24, 2022
@sosoYim sosoYim linked a pull request Jan 24, 2022 that will close this issue
10 tasks
@sosoYim sosoYim mentioned this issue Jan 25, 2022
8 tasks
@sosoYim sosoYim linked a pull request Jan 26, 2022 that will close this issue
10 tasks
@sosoYim sosoYim closed this as completed Jan 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants