Skip to content

Commit fe45702

Browse files
committed
Merge pull request #43 from colab-community/gitlab-fixes
Fixed workarounds for .hide elements
2 parents 06eb1bf + a4badbf commit fe45702

File tree

1 file changed

+5
-11
lines changed

1 file changed

+5
-11
lines changed

src/proxy/gitlab/templates/proxy/gitlab.html

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -26,17 +26,11 @@
2626
{% block head_js %}
2727
<script type="text/javascript">
2828
$(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');
4034
});
4135
</script>
4236
<script type="text/javascript" src="{% static 'third-party/bootstrap/js/bootstrap.min.js' %}"></script>

0 commit comments

Comments
 (0)