Hide Forgot
Created attachment 1144130 [details] screenshot (only unreleased develop) Random scroll bars, see attached. Unsetting overflow: auto; on the tab-content fixes them. Not sure why we have it there in the first place.
(In reply to Dan Callaghan from comment #0) > Created attachment 1144130 [details] > screenshot > > (only unreleased develop) Random scroll bars, see attached. > > Unsetting overflow: auto; on the tab-content fixes them. Not sure why we > have it there in the first place. It's because .recipe-tabs has been removed by this patch http://gerrit.beaker-project.org/#/c/4755/ The .recipe-tabs is used to set overflow:visible on the tab-content. On Gerrit: http://gerrit.beaker-project.org/#/c/4783/
Is the overflow:auto; rule coming from Bootstrap? Can we just get rid of that? When is it ever needed? Seems like we have to just keep overriding it on every tab content we have.
(In reply to Dan Callaghan from comment #2) > Is the overflow:auto; rule coming from Bootstrap? Can we just get rid of > that? When is it ever needed? Seems like we have to just keep overriding it > on every tab content we have. Yes, this rule comes from Boostrap with a comment 'prevent content from running below tabs', see line 292 in navs.less. It may be required by other boostrap components.
(In reply to matt jia from comment #1) > On Gerrit: > > http://gerrit.beaker-project.org/#/c/4783/ Repost with correct branch http://gerrit.beaker-project.org/#/c/4788/
Beaker 23.0 has been released.