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

Presenter::$layout can be either string or null #233

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

Conversation

mabar
Copy link
Contributor

@mabar mabar commented Sep 19, 2019

  • bug fix? yes
  • BC break? no

It was not really string|bool but string|false|null, which kinda surprised me when I was implementing own layout finder

This variant is the only one I found which simplifies problem and does not break default implementation. Not sure about custom ones.

Alternative variant is change of annotation to string|false|null, but it's ugly.

@dg dg added this to the v3.1 milestone Sep 24, 2019
@dg dg force-pushed the master branch 7 times, most recently from 7e42997 to f7df426 Compare October 21, 2019 19:23
@dg dg force-pushed the master branch 4 times, most recently from 0e26720 to e65cf05 Compare November 19, 2019 19:03
@dg dg force-pushed the master branch 3 times, most recently from 7fe78c1 to 8f1bb54 Compare December 11, 2019 19:32
@dg dg force-pushed the master branch 13 times, most recently from c8eff84 to 447af2d Compare January 6, 2020 12:28
@dg dg force-pushed the master branch 2 times, most recently from a1e8c9b to df7c4c9 Compare January 21, 2024 23:50
@dg dg force-pushed the master branch 2 times, most recently from 891b6dd to e3d05b3 Compare February 2, 2024 17:51
@dg dg force-pushed the master branch 3 times, most recently from 426e735 to c19ebdc Compare March 11, 2024 20:02
@dg dg force-pushed the master branch 5 times, most recently from 2b9da37 to 30d90f4 Compare April 7, 2024 02:51
@dg dg force-pushed the master branch 6 times, most recently from bf86204 to c91f90a Compare April 20, 2024 00:46
@dg dg force-pushed the master branch 3 times, most recently from 57bd587 to e908315 Compare May 2, 2024 10:37
@dg dg force-pushed the master branch 8 times, most recently from c5ecbda to ecb200c Compare May 13, 2024 09:25
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.

None yet

2 participants