Bug 532185: [Dark Theme] Help content does not respect styling
In dark appearance there are some ugly light areas. First one are
the separators between the buttons. They don't group in any meaningful
way and the SWT help view also does not have them. So let's simply
remove them.
The second one is the grey separator between the left hand side
navigation and the main content area. On today's web pages you normally
don't see them. So let's make it much thinner.
Change-Id: Ife3829693e58819d6ea391274d69c868a0bc6513
Signed-off-by: Matthias Becker <ma.becker@sap.com>
diff --git a/org.eclipse.help.webapp/advanced/contentToolbar.jsp b/org.eclipse.help.webapp/advanced/contentToolbar.jsp
index 96e8c0a..f005437 100644
--- a/org.eclipse.help.webapp/advanced/contentToolbar.jsp
+++ b/org.eclipse.help.webapp/advanced/contentToolbar.jsp
@@ -63,12 +63,6 @@
<jsp:param name="param" value="<%=homeURL%>"/>
<jsp:param name="state" value='off'/>
- <jsp:param name="name" value=""/>
- <jsp:param name="tooltip" value=""/>
- <jsp:param name="image" value=""/>
- <jsp:param name="action" value=""/>
- <jsp:param name="param" value=""/>
- <jsp:param name="state" value='off'/>
<jsp:param name="name" value="synch"/>
<jsp:param name="tooltip" value='Synch'/>
diff --git a/org.eclipse.help.webapp/advanced/help.jsp b/org.eclipse.help.webapp/advanced/help.jsp
index f929f28..5f66c1b 100644
--- a/org.eclipse.help.webapp/advanced/help.jsp
+++ b/org.eclipse.help.webapp/advanced/help.jsp
@@ -118,7 +118,7 @@
<%
}
%>
- id="helpFrameset" cols="<%=isRTL?"70.5%,29.5%":"29.5%,70.5%"%>" framespacing="3" border="6" frameborder="1" scrolling="no">
+ id="helpFrameset" cols="<%=isRTL?"70.5%,29.5%":"29.5%,70.5%"%>" framespacing="1" border="1" frameborder="1" scrolling="no">
<%
if (isRTL) {
%>
diff --git a/org.eclipse.help.webapp/advanced/tocToolbar.jsp b/org.eclipse.help.webapp/advanced/tocToolbar.jsp
index e981582..fbaaa84 100644
--- a/org.eclipse.help.webapp/advanced/tocToolbar.jsp
+++ b/org.eclipse.help.webapp/advanced/tocToolbar.jsp
@@ -46,13 +46,6 @@
<jsp:param name="action" value="menu"/>
<jsp:param name="param" value="<%=menuData%>"/>
<jsp:param name="state" value='off'/>
-
- <jsp:param name="name" value=""/>
- <jsp:param name="tooltip" value=""/>
- <jsp:param name="image" value=""/>
- <jsp:param name="action" value=""/>
- <jsp:param name="param" value=""/>
- <jsp:param name="state" value='off'/>
<jsp:param name="name" value="quick_search"/>
<jsp:param name="tooltip" value='QuickSearchMulti'/>
