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

[BUG] @prizm-ui/components - Для кнопок PrizmSidebarService нельзя установить disabled state #261

Closed
1 of 9 tasks
dkshichkovskiy opened this issue May 14, 2023 · 2 comments
Assignees
Labels
source:idp issue from IDP state:ready ready for release (pass tested)
Milestone

Comments

@dkshichkovskiy
Copy link

dkshichkovskiy commented May 14, 2023

Библиотека

  • @prizm-ui/core
  • @prizm-ui/components
  • @prizm-ui/install
  • @prizm-ui/icons
  • @prizm-ui/flag-icons
  • @prizm-ui/theme
  • @prizm-ui/charts
  • @prizm-ui/ast
  • @prizm-ui/nx-plugin

Компонент

SideBar

Версия

1.0.0-rc.1

Описание проблемы

При прокидывании кнопок в вызов метода sidebarService.open нельзя задать условия, при которых блокируются эти кнопки. В настоящий момент типизация позволяет задать поле disabled (с типом boolean), но его значение по факту игнорируется.

this.sidebarService
  .open(new PolymorphComponent(SomeComponent, this.injector), {
    ...
    confirmButton: {
      text: 'Button'
      disabled: true,
    },
  });

Хотелось бы иметь возможность определять условия блокировки таких кнопок: чтобы поле disabled не игнорировалось и в качестве его значения можно было бы указать Observable<boolean>.

@irustm irustm added the source:idp issue from IDP label May 15, 2023
@skkonstantin skkonstantin self-assigned this May 16, 2023
@skkonstantin skkonstantin added the state:in_progress We work on this issue label May 16, 2023
skkonstantin pushed a commit that referenced this issue May 16, 2023
@skkonstantin skkonstantin added state:ready ready for release (pass tested) and removed state:in_progress We work on this issue labels May 17, 2023
@ZurabDev ZurabDev added this to the 1.0.0-rc.2 milestone May 19, 2023
@dkshichkovskiy
Copy link
Author

dkshichkovskiy commented May 25, 2023

@ZurabDev Свойство disabled заработало, спасибо.

В описании issue так же идёт речь про тип Observable<boolean> для этого свойства. Можно ли такую возможность добавить?

@ZurabDev
Copy link
Contributor

@dkshichkovskiy списобо @skkonstantin это он делал. А по поводу Observable можешь создать с этими issues плиз

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
source:idp issue from IDP state:ready ready for release (pass tested)
Projects
None yet
Development

No branches or pull requests

4 participants