Skip to content

Commit

Permalink
Dev: used {{ thissurvey.name }} instead of {SURVEYNAME}
Browse files Browse the repository at this point in the history
  • Loading branch information
LouisGac committed Dec 21, 2016
1 parent c9213dd commit 30e9e05
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion templates/default/views/startpage.pstpl
View file
Expand Up @@ -46,7 +46,7 @@
<span class="icon-bar"></span>
<span class="icon-bar"></span>
</button>
<div class="navbar-brand" >{SURVEYNAME}</div>
<div class="navbar-brand" >{{ thissurvey.name }}</div>

This comment has been minimized.

Sign in to view

Copy link
@olleharstedt

olleharstedt Dec 23, 2016

Contributor

How about {survey.name}}? Will it still work without the spaces inside the brackets?

This comment has been minimized.

Sign in to view

Copy link
@LouisGac

LouisGac Jan 4, 2017

Contributor

yes
</div>
<div id="navbar" class="collapse navbar-collapse">
<ul class="nav navbar-nav navbar-right navbar-action-link">
Expand Down

3 comments on commit 30e9e05

@tpartner
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see a commented list of the TWIG vars in the .pstpl file but do we have a list of them (and their descriptions) in the manual anywhere?

@LouisGac
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tpartner: we still have to choose what variables will be available from the pstpl file. But:

  1. A good guess is: oSurvey, oQuestionGroup, and oQuestion
  2. While moving the logic from the helpers to the pstpl file, we'll find all the needed variables to pass.

@tpartner
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

Please sign in to comment.