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.

Sign up for GitHub

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

Jump to bottom

Прокачать главную страницу #220

Closed
fey opened this issue Apr 21, 2020 · 18 comments
Closed

Прокачать главную страницу #220

fey opened this issue Apr 21, 2020 · 18 comments
Labels
help wanted Extra attention is needed

Comments

@fey
Copy link
Collaborator

fey commented Apr 21, 2020 •
edited
Loading

Можно добавить на главную страницу различную статистику:
Оставленных комментариев, пройденных упражнений, глав. Сколько человек выполнили все задания и главы.
@fey fey changed the title Статистика на главную страница Статистика на главную страницу May 12, 2020
@fey fey changed the title Статистика на главную страницу Прокачать главную страницу Jun 25, 2020
@fey
Copy link
Collaborator Author

fey commented Jun 25, 2020

feycot:banana: 14:14
Привет, там не то что бы объемно, просто комплексная проблема с переделкой главной странички
14:15
Т.е. можно в уголок зафигарить блок и туда добавить типа "сегодня решили N упражнений, оставили N комментариев, N всего осиляторов"
14:16
Т.е. по сути несколько запросов к бд составить и циферки вывести на страницу. Но! Надо подумать как лучше это сделать. Потому что ща статистика находится на нижней части страницы, т.е. её нужно скросллить, чтобы увидеть ту статистику и активность, что есть
14:18
Как вариант можно уменьшит отступ от навбара и h1 чтобы поднять контент, поменять сетку, чтобы описание-подзаголовок ушел влево, поднять боковую сторону и вставить туда статистику :thinking_face:

@fey
Copy link
Collaborator Author

fey commented Jul 17, 2020

feycot:pepe_hype: 3 days ago
https://hexlet-sicp-test.herokuapp.com/ru
https://sicp.hexlet.io/ru
Ребзи, у нас тут навигация тюнится в сикп. Потыкайте, посмотрите, понятно ли что куда нажимать. Пишите в тред если че (edited)
💯
1

Glagius 3 days ago
Крутые перцы!
На все руки мастера. Свои Руби / Питон / Ноду пишут.
image.png
image.png

Glagius 3 days ago
Может просто названия проектов и в скобках стек?
image.png
image.png

feycot:pepe_hype: 3 days ago
Ну можно, присылай пулл реквест :batya:

Glagius 3 days ago
ПхП не моё 🙂

feycot:pepe_hype: 3 days ago
Там ток текст поменять, можно даже проект не разворачивать, а прямо на гитхабе поправить

Glagius 3 days ago
И еще момент - может сразу тыкнуть пальтсем, ЧТО именно меняется? Я посмотрел навигацию внизу / наверху, вроде проблем не возникло, но хз что именно хотели проврить.

Glagius 3 days ago
Там ток текст поменять, можно даже проект не разворачивать, а прямо на гитхабе поправить
Это скучно 🙂

feycot:pepe_hype: 3 days ago
сейчас например менюшка вложена в дропдаун (сверху слева), а будет как нормальная навигация

Glagius 3 days ago
Мне как стороннему человеку непонятно назначение этих зелёных кубиков. Вроде в гитхабе так показывают МОЮ активность. Но я не зареган.
image.png
image.png

feycot:pepe_hype: 3 days ago
image.png
image.png

Glagius 3 days ago
А я картинок накидаю 🙂
image.png
image.png

feycot:pepe_hype: 3 days ago
Что еще можно добавить?

Glagius 3 days ago
Призыва "отметить прочитанное" нет. Только моё любопытство сподвигло тыкнуть на кнопку. И всё равно попал на регистрацию.
image.png
image.png

feycot:pepe_hype: 3 days ago
@mshkv Андрей, призови, пожалуйста, Алексея, чтобы если заскучает посмотрел на наше безобразие

mshkv:hexlet: 3 days ago
@alshad призвал

Glagius 3 days ago
https://hexlet-sicp-test.herokuapp.com/ru/chapters
Я бы на этой странице, справа / слева добавил блок с оглавнением.
Громадный список с главами вгоняет в уныние, не хочется прокручивать вниз.

Glagius 3 days ago
Либо сделать главы сворачивающимися, потому что там текста и нет толком, только список ссылок.

Glagius 3 days ago
Кто эти люди? Кто стащил запятые?
image.png
image.png

Glagius 3 days ago
Зачем мне прокручивать вниз, чтобы узнать что есть добавленные упражнения?
image.png
image.png

Glagius 3 days ago
Если меняю локаль, меня сбрасывает на основной роут.

feycot:pepe_hype: 3 days ago
Зачем мне прокручивать вниз, чтобы узнать что есть добавленные упражнения?
это как (edited)

Glagius 3 days ago
Выше я показал. У меня весь список упражнений не влазит на страницу, а если сделать масштаб больше, то тем более не влезет. Да и уйма "пустых" мест явно характеризует "недоделаность" проекта.

Glagius 3 days ago
Нужны ли ссылки на несуществующие упражнения?
И как его кто-то завершит, если его нет?))
image.png
image.png

feycot:pepe_hype: 3 days ago
Просто упражнение можно отметить прочитанным, но текста пока нет. Там ссылка на сайт мит есть. Т.е. можно открыть упражнение, решить

Glagius 3 days ago
Может тогда ссылку сделать ссылкой а не иконкой? Я вот хз, это оформление такое или реально ссылка на сторонний ресурс.

Glagius 3 days ago
Пока так. Пойду, дальше поработаю.

feycot:pepe_hype: 3 days ago
да, можно вместо "нет описания" вставить ссылку на мит @alshad

feycot:pepe_hype: 3 days ago
Спасибо, фидбек полезен (edited)
:cat_typing:
1

Serhii Shramko 3 days ago
https://tppr.me/8AuhM про кнопку согласен 100. Не сразу понял, как начать.
Потому как я думал это, что то типа курсов и искал кнопку начать. Ведь я ще ничего не читал и про СИКП не знаю
tppr.metppr.me
Public Screenshots - Screenshot_2020-07-14 10-08-51 (262 kB)
https://tppr.s3.amazonaws.com/uploads/515e8fb7a9bab162a4c805fbf45ed0b4.jpg

Serhii Shramko 3 days ago
Начать/Начать обучение/Начать прохождение что-то в этом роде было бы лучше имхо

Evgeny Malafeev 3 days ago
Эльдар разошёлся)

Glagius 3 days ago
Та неее, просто ребятам помочь захотел 🙂

@fey
Copy link
Collaborator Author

fey commented Jul 17, 2020

image

image

image
"Зачем мне прокручивать вниз, чтобы узнать что есть добавленные упражнения?"
image

@fey
Copy link
Collaborator Author

fey commented Jul 17, 2020

"Зачем мне прокручивать вниз, чтобы узнать что есть добавленные упражнения?"
image

@acidmaksim acidmaksim added the help wanted Extra attention is needed label Sep 13, 2022
@ashikov
Copy link
Contributor

ashikov commented Nov 8, 2022

@acidmaksim тут бы сделать краткие конспекты из диалогов, чтобы было проще войти в контекст

@ashikov
Copy link
Contributor

ashikov commented Nov 15, 2022

@acidmaksim

@acidmaksim
Copy link
Contributor

@ashikov сегодня сделаю

@AslanAV
Copy link
Contributor

AslanAV commented Nov 21, 2022

Привет, я сделал примерно
https://hexlet-sicp-aslan-av.herokuapp.com/
доступно залогиненным пользователям.

Для гостя выводится другая страница, и там другое оформление, там есть карусель,
есть другая статистика

  • 129 chapters
  • 356 exercises
  • 12 users
  • 29 comments

по гостевой надо больше уточнений, что убрать

@fey
Copy link
Collaborator Author

fey commented Nov 21, 2022

Для гостя по сути выводится лендинг, который должен приглашать его зарегистрироваться и начать обучение.
На самом деле в случае лендинга надо подумать о том. как сделать супер-пупер лендинг.

@fey
Copy link
Collaborator Author

fey commented Nov 21, 2022

@acidmaksim напоминалка про то, чтобы описать идеи для главной

@acidmaksim
Copy link
Contributor

@fey мы с Асланом уже все обсудили, он скинул прототип

@acidmaksim
Copy link
Contributor

@AslanAV ты немного не так сделал. Нужно было поднять весь текст на первом экране лендинга, а в появившемся месте, как раз добавить статистику

@fey
Copy link
Collaborator Author

fey commented Nov 21, 2022

В ишшусе-то контекста нет)
Точнее этот ишшус - это обсуждение. А так если надо что-то конкретное сделать, можно создать новый с техническим заданием

@AslanAV
Copy link
Contributor

AslanAV commented Nov 21, 2022

точно, нужно, а то у меня плывет все в голове :)
https://hexlet-sicp-aslan-av.herokuapp.com/

подправил статистику, раньше выводило общее количество всех заданий и упражнений)))), сейчас отмеченные как пройденные и решенные, выводятся на главную

@fey
Copy link
Collaborator Author

fey commented Nov 21, 2022

image
текст в карусели не вмещается.

@fey
Copy link
Collaborator Author

fey commented Nov 21, 2022

По лендингу. Нам делали новый дизайн, нужно его кстати использовать - https://github.com/Hexlet/hexlet-sicp/tree/main/design
Но минус в том, что щас кнопка призыва к действию находится на 2 экране. Можно попробовать своими силами это поправить

AslanAV added a commit to AslanAV/hexlet-sicp that referenced this issue Nov 27, 2022
@AslanAV
[Hexlet#220] add locale translate
50e9f17
@AslanAV AslanAV mentioned this issue Nov 27, 2022
Merged
AslanAV added a commit to AslanAV/hexlet-sicp that referenced this issue Nov 27, 2022
@AslanAV
[Hexlet#220] fix phpstan: add type for parameter $filter in HomeContr…
82f1aa2 
…oller.php method getStatisticTable()
fey added a commit that referenced this issue Dec 12, 2022
@fey
Merge pull request #1415 from AslanAV/issue/new-up-main-page-#220
01d6a83 
Issue #220 / add statistic table to main page
@ssssank
Copy link
Contributor

ssssank commented May 3, 2023

@fey а здесь ещё не всё?

@fey
Copy link
Collaborator Author

fey commented May 3, 2023

Да, думаю можно закрывать.

@ssssank ssssank closed this as completed May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@ashikov @fey @ssssank @AslanAV @acidmaksim