Skip to content

Refactor dialogs + drawers as classes #6967

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

Draft
wants to merge 12 commits into
base: v6/develop
Choose a base branch
from

Conversation

distantnative
Copy link
Member

@distantnative distantnative commented Feb 2, 2025

Description

Todo

  • Decide on term for handler key

Summary of changes

  • Backend PHP Panel\Ui classes for all dialogs and drawers
  • Dialog and drawer routes: support passing a handler closure that returns an object with ::render() and ::submit() methods that will be used then as load/submit event actions

Additional context

PageCreateDialog is a bit more complex and tricky and would be refactored in a separate step.

Ready?

  • In-code documentation (wherever needed)
  • Unit tests for fixed bug/feature
  • Tests and CI checks all pass

For review team

  • Add changes & docs to release notes draft in Notion

@distantnative distantnative self-assigned this Feb 2, 2025
@distantnative distantnative force-pushed the v5/dialogs/1-backend-classes branch from 435529c to 76d283c Compare February 2, 2025 12:55
@distantnative distantnative force-pushed the v5/dialogs/1-backend-classes branch 2 times, most recently from 5ffebf3 to 22eaf57 Compare February 2, 2025 12:59
@distantnative distantnative force-pushed the v5/dialogs/1-backend-classes branch from 4d3c5f7 to a8e6f68 Compare February 7, 2025 14:04
@distantnative distantnative force-pushed the v5/dialogs/1-backend-classes branch from d5ee59a to d767771 Compare February 8, 2025 17:35
@distantnative distantnative marked this pull request as ready for review February 8, 2025 18:04
@distantnative distantnative marked this pull request as draft February 10, 2025 09:19
@distantnative distantnative added this to the 6.0.0-alpha.1 milestone Feb 10, 2025
@distantnative distantnative changed the base branch from v5/develop to v6/develop February 10, 2025 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant