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

Таблица "Рейтинг Акций Мозговик" #19

Closed
BDemetrix opened this issue Aug 11, 2023 · 6 comments
Closed

Таблица "Рейтинг Акций Мозговик" #19

BDemetrix opened this issue Aug 11, 2023 · 6 comments

Comments

@BDemetrix
Copy link
Owner

No description provided.

@kulbergArt
Copy link

kulbergArt commented Aug 11, 2023

У нас есть платная аналитика, и там у нас есть табличка с рейтингом по компаниям.

Вова её попытался собрать, но там страхолюдище получилось. Тим просит сделать нормально. Я в Фигме нарисовал, как это выглядит, но по идее бы использовать уже созданный тобой дизайн.

sMart-lab.ru - рейтинг акций Мозговик
Чтобы посмотреть таблицу, войди под моим логином и паролем

Дизайн: Artem Smart-lab

image

@kulbergArt
Copy link

kulbergArt commented Aug 11, 2023

Дим, там были пожелания Тима, и я их отразил в дизайне, но забыл написать в ТЗ:

  1. Закрепляться должен только столбик с тикерами. Столбик с названием компаний должен идти второй, и при скролле скрываться.

  2. Ширина столбцов по возможности одинаковая, как в дизайне. Когда ширина зависит от шапки столбца, получается не красиво.

  3. Убери, пожалуйста, этот пустой столбец.

  4. Последние два столбца, скорее всего должны быть с иконками, тут подождём что Тим скажет.

@kulbergArt
Copy link

kulbergArt commented Aug 11, 2023

иконка ссылки, тим её утвердил

Image

@BDemetrix
Copy link
Owner Author

комменатий для Вовы

PR привязаны к задаче
DEMO https://bdemetrix.github.io/sl_mobile/build/stocks-rating.html

Дописал изменения в скрипт для таблицы. Теперь если надо будет задать минимальную ширину колонки (лучше минимальную, чтоб динамический контент не ломал ветстку), надо прописать самой верхней ячейке задать ее в стилях инлайн, например: <th style="min-width: 100px;"> . класс для всех ячеек этой колонки для ширины теперь не нужен

@kulbergArt
Copy link

Дим, нужно ещё небольшое обновление сделать:
Актуальный вид таблицы сделал в дизайне.

Image

@BDemetrix
Copy link
Owner Author

BDemetrix commented Aug 17, 2023

Актуальный вид таблицы сделал в дизайне.

Артем, на такие дополнения пиши пожалуйста тикет, это требует времени и возможно будет сделано не сразу, а в комментах либо потеряется либо задача в основной части будет висеть.
Сейчас таблицу Вова уже может исправить.
А раскрашу потом.

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