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

PLASMA-4117: PaginationDots api & example #1714

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Conversation

vadim-kudr
Copy link
Contributor

@vadim-kudr vadim-kudr commented Jan 22, 2025

PaginationDots

FIGMA

EXAMPLE

API, пример в коммите


interface Props {
  size?: 's' | 'm';
  /**
   * Отображение активного элемента
   * @default 'dot'
   */
  activeElementView?: 'dot' | 'line';
  /**
   * Ориентация элементов
   * @default `horizontal`
   */
  orientation?: 'horizontal' | 'vertical';
  /**
   * Растягивается на всю доступную ширину
   * @default true
   */
  stretch?: boolean;

  /**
   * Выбранная страница
   */
  value?: number;

  /**
   * Количество страниц
   */
  count?: number;
}

Copy link
Contributor

Theme Builder app deployed!

https://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-1714/

@vadim-kudr vadim-kudr marked this pull request as ready for review January 22, 2025 07:06
@github-actions github-actions bot requested review from iljs and neretin-trike and removed request for Yeti-or, Yakutoc, neretin-trike, TitanKuzmich and shuga2704 January 22, 2025 07:06
@Yakutoc Yakutoc requested review from shuga2704 and removed request for iljs January 29, 2025 08:22
@Yakutoc Yakutoc changed the title feat(*): PaginationDots api & example PLASMA-4117: PaginationDots api & example Jan 29, 2025
@shuga2704
Copy link
Contributor

  1. Если поставить count = 20, и value тоже 20, то точка не отобразится (выглядит как баг примера).
  2. Мб не stretch, а stretched?

@neretin-trike
Copy link
Collaborator

Мб не stretch, а stretched?

блин, правильнее конечно как ты предложил, правда у нас уже есть "stretch" 🤔 Предлагаю тоже делать "stretched"

* Количество страниц
*/
count?: number;
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

  1. А у пользователя не должно быть возможности кликать на эти точки чтобы изменить активное состояние?
  2. А если будет условно 90 элементов, тогда все 90 точек будем выводить? Для такого подхода не должен быть предусмотрен механизм "сжатия" или это на стороне пользователя?

Copy link
Collaborator

Choose a reason for hiding this comment

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

ещё кажется, что не хватает такого свойства

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ещё кажется, что не хватает такого свойства

это для сторибука

Copy link
Contributor Author

Choose a reason for hiding this comment

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

про onClick спрошу у дизайна

Copy link
Contributor Author

Choose a reason for hiding this comment

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

onClick опционально будет

@vadim-kudr
Copy link
Contributor Author

stretch

по проекту используется stretch, stretched нигде нет. исходил из этого

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants