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

Inside a call to dashboardPage(), correctly extract title from dashboardHeader() #203

Merged
merged 2 commits into from Apr 20, 2017

Conversation

@bborgesr
Copy link
Contributor

@bborgesr bborgesr commented Apr 20, 2017

This closes #89. We claimed that dashboardPage() would try to extract the app's title from dashboardHeader() (if the title is not provided directly to dashboardPage()); however, we were selecting the wrong child of the header tag object.

This must have worked at some point and then some change to the structure of the header object broke it.

…e's title from dashboardHeader (if the title is not provided directly to dashboardPage); however, we were selecting the wrong child of the header tag object
@wch wch merged commit d4a64e4 into master Apr 20, 2017
4 checks passed
4 checks passed
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@wch wch removed the in progress label Apr 20, 2017
@wch wch deleted the barbara/extract-title branch Apr 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.