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

feat(components/paginator): add left direction support #1154 #1886

Merged
merged 5 commits into from
Sep 16, 2024

Conversation

ickisIckis
Copy link
Collaborator

@ickisIckis ickisIckis commented Jul 19, 2024

Библиотека

  • @prizm-ui/components
  • @prizm-ui/install
  • @prizm-ui/icons
  • @prizm-ui/theme

Компонент

Paginator

Задача

resolved #1154

Изменения

  • Имеются BREAKING CHANGES
  • Изменения документации
  • Добавление фичи
  • Исправление бага

Checklist:

  • После фичи обновил документацию
  • Сделал код чище чем был до этого
  • Тесты и линтер на рабочей машине успешно выполнились

Следует обратить внимание на ревью

Реализация через boolean, возможно стоит сделать через left | right, где right - дефолтное?

Следует обратить внимание при тестировании

Верстка пагинатора с левым и правым расположением

Release Notes

Реализовали вариант расположения элементов управления в пагинаторе - слева направо.

Copy link

github-actions bot commented Jul 19, 2024

Visit the preview URL for this PR (updated for commit fa13510):

https://prizm-v4--pr1886-feat-paginator-left-38vqhrwj.web.app

(expires Mon, 12 Aug 2024 10:21:06 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 7c62ed8dbabf5e2d6b2084ca9e107cc206d30dbd

@Input()
@HostBinding('class.direction-left')
@prizmDefaultProp()
public directionLeft = false;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Может добавить direction c по умалчанием значением right

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Да. отлично. Так и сделаем.

Copy link
Contributor

@ZurabDev ZurabDev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alexhawkins94
Copy link

Для левой раскладки не хватает margin left для "Показано строк".
chrome_Vo73Xfr5af

@ickisIckis ickisIckis added the state:ready ready for release (pass tested) label Aug 8, 2024
@ickisIckis ickisIckis changed the base branch from main to v4 August 23, 2024 08:31
@ZurabDev ZurabDev merged commit d6169da into v4 Sep 16, 2024
9 of 10 checks passed
@ZurabDev ZurabDev mentioned this pull request Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Paginator. Добавить расположение элементов Left
3 participants