Skip to content

feat: add $mol_url#817

Open
vladislav-atakhanov wants to merge 1 commit intohyoo-ru:masterfrom
vladislav-atakhanov:mol_url
Open

feat: add $mol_url#817
vladislav-atakhanov wants to merge 1 commit intohyoo-ru:masterfrom
vladislav-atakhanov:mol_url

Conversation

@vladislav-atakhanov
Copy link
Copy Markdown

Добавил $mol_url - текстовое поле с валидацией URL

Ещё сомневаюсь, как сделать не обязательным (сейчас check присваивается null), вообще если не обязательно, то лучше $mol_string использовать

Можно добавить поля, которые будут передаваться в $mol_string

Fixes #28

@vladislav-atakhanov
Copy link
Copy Markdown
Author

Ещё типы не сходятся, хотя в js всё хорошо

Для message должен тип string | Record<string, string>, но выдает ошибку Type 'string' does not satisfy the constraint 'Record<string, string>'

Ещё $mol_button_minor не совместим с $mol_check_icon, который требует $mol_password

@zerkalica
Copy link
Copy Markdown
Collaborator

zerkalica commented Dec 12, 2025

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

Для внешней ссылки можно взять https://mol.hyoo.ru/#!section=demos/demo=mol_link_iconed_demo а не делать ссылку из кнопки зачем-то.

Валидаторы лучше делать небольшими псевдокомпонентами отдельно и композировать в форме: https://github.com/zerkalica/yuf/blob/fcda8af53015009139a83d8134d9b381baeb8a73/login/form/form.view.tree#L5

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

Successfully merging this pull request may close these issues.

Url

2 participants