Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(476)

Side by Side Diff: app/templates/service-configuration.handlebars

Issue 10964044: Add expose/unexpose to the inspector.
Patch Set: Created 10 years, 8 months ago
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 <div class="view-container"> 1 <div class="view-container">
2 <h2>Expose</h2>
3 <div class="control-expose control">
4 <div class="inline">
5 <span>Expose this service?</span>
6 </div>
7 <div class="toggle-expose" data-bind="exposed">
8 <img src="" />
9 <span></span>
jeff.pihach 2013/07/05 19:39:13 Should these not have sane defaults?
benji 2013/07/05 19:40:02 These two lines could use some comments explaining
bac 2013/07/05 20:57:07 Probably.
10 </div>
11 </div>
12
2 <h2>Service Settings</h2> 13 <h2>Service Settings</h2>
14 <hr style="border: dashed; border-width: 1px 0 0 0">
jeff.pihach 2013/07/05 19:39:13 Is it intended that this will be removed?
benji 2013/07/05 19:40:02 I don't feel this way, but the project as a whole
bac 2013/07/05 20:57:07 Yes, it should be removed. I was experimenting to
3 <div class="config-file"> 15 <div class="config-file">
4 <input type="file"> 16 <input type="file">
5 <div class="fakebutton">Import config file...</div> 17 <div class="fakebutton">Import config file...</div>
6 </div> 18 </div>
7 <div class="toggle-settings-help">Hide settings help</div> 19 <div class="toggle-settings-help">Hide settings help</div>
8 {{> service-configuration}} 20 {{> service-configuration}}
9 </div> 21 </div>
OLDNEW
« no previous file with comments | « [revision details] ('k') | app/views/environment.js » ('j') | app/views/environment.js » ('J')

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b