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

Add @method make phpDoc to Tabs component #1532

Closed

Conversation

shevelev-anatoliy
Copy link
Contributor

  • Tested
    • Tested manually
    • Tests added
  • Documentation

@shevelev-anatoliy shevelev-anatoliy changed the title Add @method make to Tabs component Add @method make phpDoc to Tabs component Feb 9, 2025
@lee-to
Copy link
Collaborator

lee-to commented Feb 9, 2025

Я в данном случае специально не делал докблок, так как свойство items нужно для блейд компонента, а для класса с items лучше не работать и не видеть

@lee-to lee-to closed this Feb 9, 2025
@shevelev-anatoliy
Copy link
Contributor Author

shevelev-anatoliy commented Feb 9, 2025

Я в данном случае специально не делал докблок, так как свойство items нужно для блейд компонента, а для класса с items лучше не работать и не видеть

В доке тоже прописана полная сигнатура метода make() с items, но там нет смысла убирать, иначе проще вообще убрать сигнатуру метода

@lee-to
Copy link
Collaborator

lee-to commented Feb 9, 2025

Я в данном случае специально не делал докблок, так как свойство items нужно для блейд компонента, а для класса с items лучше не работать и не видеть

В доке тоже прописана полная сигнатура метода make() с items, но там нет смысла убирать, иначе проще вообще убрать сигнатуру метода

думаю и в доке можно убрать

@shevelev-anatoliy
Copy link
Contributor Author

Я в данном случае специально не делал докблок, так как свойство items нужно для блейд компонента, а для класса с items лучше не работать и не видеть

В доке тоже прописана полная сигнатура метода make() с items, но там нет смысла убирать, иначе проще вообще убрать сигнатуру метода

думаю и в доке можно убрать

Готово

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.

2 participants