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
EZP-27501: use pjax as a pagelayout #21
Conversation
7a64779
to
827c026
Compare
827c026
to
e9d847b
Compare
d750253
to
31c333e
Compare
31c333e
to
486c2af
Compare
ebdb6a7
to
a4d3280
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FWIW
/** | ||
* Matches a Pjax Http Response. | ||
*/ | ||
interface PjaxResponseMatcher extends ResponseMatcherInterface |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is one named -Interface and the other not?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah... 'cause I wanted ResponseMatcherInterface
to be consistent with Symfony's RequestMatcherInterface
. Do you see my dilemma ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's follow the trend, and go for Facey McInterface-face.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 besides the default title comment.
not strictly part of this story, we should check why we have a blank page after login or when providing wrong credentials.
@@ -0,0 +1,15 @@ | |||
<div data-name="title">{% block title %}{{ title|default('hybrid pagelayout title') }}{% endblock %}</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the default title should probably be an empty string
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed.
a4d3280
to
0050219
Compare
Configures
src/bundle/Resources/views/pagelayout.html.twig
as the global pagelayout for theadmin_group
siteaccess group. Core templates that use it, `such as content_edit.html.twig, will be rendered within the PJAX DOM and integrated in the Hybrid UI.In addition, adds a
PjaxResponseMatcher
, used to detect PJAX responses by looking for<div data-name="html">
in the content. It allows to match requests that don't identify by the URI or by the PJAX header.TODO