Fix JavaScript alert() about missing search terms when submitting patch review
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 16 Oct 2014 16:01:37 +0000 (16:01 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 16 Oct 2014 16:01:37 +0000 (16:01 +0000)
* template/en/default/attachment/reviewform.html.tmpl: Pass
hide_content=1 argument when including global/header.html.tmpl.
* template/en/default/global/header.html.tmpl: Add hide_content argument
that hides header content when including the file.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@174785 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Websites/bugs.webkit.org/template/en/default/attachment/reviewform.html.tmpl
Websites/bugs.webkit.org/template/en/default/global/header.html.tmpl

index 873d902..d1194a8 100644 (file)
@@ -25,7 +25,6 @@
 #comment { width: 100%; height: 100%; font-family: monospace; }
 #form-controls { height: 40px; }
 #form-controls td { width: 25%; text-align: center; }
-#header { display: none; }
 </style>
 
 [% javascript = BLOCK %]
@@ -55,6 +54,7 @@
 [% END %]
 
 [% PROCESS global/header.html.tmpl
+  hide_content = 1
   javascript = javascript
 %]
 
index 6fa6b85..56f05ce 100644 (file)
@@ -35,6 +35,7 @@
   # style_urls: list. List of URLs to CSS style sheets.
   # message: string. A message to display to the user. May contain HTML.
   # rsslink: RSS link URL, May contain HTML (WEBKIT_CHANGES)
+  # hide_content: boolean. Turns off content if not being used. (WEBKIT_CHANGES)
   #%]
 
 [% IF message %]
   # 'bannerhtml'
   #%]
 
+[% IF !hide_content %][%# WEBKIT_CHANGES #%]
+
 <div id="header">
 
 [%# if WEBKIT_CHANGES #%]
 [% PROCESS "global/common-links.html.tmpl" qs_suffix = "_top" %]
 </div> [%# header %]
 
+[% END %][%# !hide_content #%][%# WEBKIT_CHANGES #%]
+
 <div id="bugzilla-body">
 
 [% IF Param('announcehtml') %]