Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: templates/minimal.tmpl

Issue 29720642: Noissue - Added .body-container to provide padding-top for body content and avoid navbar overlap (Closed) Base URL: https://hg.adblockplus.org/web.adblockplus.org
Patch Set: Created March 12, 2018, 5:02 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« templates/default.tmpl ('K') | « templates/default.tmpl ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: templates/minimal.tmpl
===================================================================
--- a/templates/minimal.tmpl
+++ b/templates/minimal.tmpl
@@ -27,16 +27,18 @@
{% block head %}
{{ head | safe }}
{% endblock %}
</head>
<body>
<? include navbar ?>
+ <div class="body-container">
juliandoucette 2018/03/12 17:08:19 NIT: I think that this could/should be an ID to be
ire 2018/03/13 08:31:36 I think it should be an ID too. Also to enforce th
juliandoucette 2018/03/13 12:14:04 Acknowledged.
{% block body %}
{{ body | safe }}
{% endblock %}
+ </div>
<? include footer ?>
<? include scripts ?>
</body>
</html>
« templates/default.tmpl ('K') | « templates/default.tmpl ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld