Fixed workarounds for .hide elements

Charles Oliveira
1 parent 988be0a3
Showing 1 changed file with 5 additions and 11 deletions Show diff stats
src/proxy/templates/proxy/gitlab.html
@@ -26,17 +26,11 @@
 {% block head_js %}
 <script type="text/javascript">
   $(function(){
-    // TODO: Somehow when there's a merge request, all options just disapear
-    // so this fix makes it show up again
-    $('div.panel-body div.automerge_widget').removeClass('hide');
-
-    // TODO: Somehow the dangerous settings are not showing up for default
-    // but for a mistery, they come back to live if the first status is
-    // not-hide
-    $('div.danger-settings div.js-toggle-content').removeClass('hide');
-
-    // TODO: For the same reason from the two walkarounds above
-    $('div.new-group-member-holder').removeClass('hide');
+    // bootstrap.css forces .hide {display:none!important}, and this makes
+    // gitlab .hide elements NEVER have a display:block, so
+    // instead of editing bootstrap.css, we just removed '.hide' css class and
+    // toggled 
+    $('.hide').removeClass('hide').css('display', 'none');
   });
 </script>
 <script type="text/javascript" src="{% static 'third-party/bootstrap/js/bootstrap.min.js' %}"></script>
...	...	@@ -26,17 +26,11 @@
26	26	{% block head_js %}
27	27	<script type="text/javascript">
28	28	$(function(){
29		- // TODO: Somehow when there's a merge request, all options just disapear
30		- // so this fix makes it show up again
31		- $('div.panel-body div.automerge_widget').removeClass('hide');
32		-
33		- // TODO: Somehow the dangerous settings are not showing up for default
34		- // but for a mistery, they come back to live if the first status is
35		- // not-hide
36		- $('div.danger-settings div.js-toggle-content').removeClass('hide');
37		-
38		- // TODO: For the same reason from the two walkarounds above
39		- $('div.new-group-member-holder').removeClass('hide');
	29	+ // bootstrap.css forces .hide {display:none!important}, and this makes
	30	+ // gitlab .hide elements NEVER have a display:block, so
	31	+ // instead of editing bootstrap.css, we just removed '.hide' css class and
	32	+ // toggled
	33	+ $('.hide').removeClass('hide').css('display', 'none');
40	34	});
41	35	</script>
42	36	<script type="text/javascript" src="{% static 'third-party/bootstrap/js/bootstrap.min.js' %}"></script>
...	...