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

Web UI: PageHeader react component with main navigation #12

Closed
sergeyt opened this issue Oct 17, 2016 · 5 comments
Closed

Web UI: PageHeader react component with main navigation #12

sergeyt opened this issue Oct 17, 2016 · 5 comments
Assignees

Comments

@sergeyt
Copy link
Contributor

sergeyt commented Oct 17, 2016

This component will be used across all pages.

Navigation links:

  • Calendars
  • Reports
  • Admin

And user avatar with dropdown menu with the following actions:

  • Profile
  • Logout
@sergeyt sergeyt changed the title Page header with main navigation PageHeader component with main navigation Oct 17, 2016
@sergeyt sergeyt changed the title PageHeader component with main navigation Web UI: PageHeader react component with main navigation Oct 17, 2016
@PolarWooolf PolarWooolf self-assigned this Oct 21, 2016
@PolarWooolf
Copy link
Contributor

Эти ссылки должны быть фиксированно только такими или сделать массив, откуда они будут подтягиваться? Также делать ли проверку на права и залогиненность пользователя (хотябы зачатки)?

@PolarWooolf
Copy link
Contributor

И еще: где и как мы должны добавлять к странице PageHeader? В index.tsx аналогично Root (или, может, как-то по-другому?) или на каждой странице отдельно если он там нужен?

@sergeyt
Copy link
Contributor Author

sergeyt commented Oct 26, 2016

@CodersGit

Эти ссылки должны быть фиксированно только такими или сделать массив, откуда они будут подтягиваться?

делай массив, можешь пока харкодить в компоненте

Эти ссылки должны быть фиксированно только такими или сделать массив, откуда они будут подтягиваться?

отдельная задача, можешь завести и делать или другу предложи :)

И еще: где и как мы должны добавлять к странице PageHeader? В index.tsx аналогично Root (или, может, как-то по-другому?) или на каждой странице отдельно если он там нужен?

пока можно копировать, потом зарефакторим с помощью react-router когда появится много страниц

@PolarWooolf
Copy link
Contributor

Осталась бага, почему-то менюшка с никнеймом остается слева. также текстовое поле в массиве container пушится после него, хотя в коде это происходит, наоборот, до этого. Почему так - не знаю.

@sergeyt
Copy link
Contributor Author

sergeyt commented Oct 31, 2016

@CodersGit создавай PR, на багу заведи ишуй. Сейчас главное функциональность. Баги потом набросимся поправим.

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

No branches or pull requests

2 participants