Commit 0f9149c4 authored by Tim Zallmann's avatar Tim Zallmann

Merge branch '14684-security-dashboard-component-replace-main-tag-with-article-tag' into 'master'

Resolve "Security Dashboard Component: Replace main-tag with article-tag"

Closes #14684

See merge request gitlab-org/gitlab-ee!16031
parents b600fd92 ef4fb63e
......@@ -148,12 +148,12 @@ export default {
<vulnerability-count-list v-if="shouldShowCountList" class="mb-0" />
<div class="row mt-4">
<main role="main" class="col" :class="{ 'col-xl-7': !isLockedToProject }">
<article class="col" :class="{ 'col-xl-7': !isLockedToProject }">
<security-dashboard-table
:dashboard-documentation="dashboardDocumentation"
:empty-state-svg-path="emptyStateSvgPath"
/>
</main>
</article>
<aside v-if="shouldShowChart" class="col-xl-5">
<vulnerability-chart />
......
......@@ -40,9 +40,7 @@ describe 'Group overview', :js do
it 'displays the Security Dashboard view' do
visit group_path(group)
page.within(find('main')) do
expect(page).to have_selector('.js-security-dashboard-table')
end
expect(page).to have_selector('.js-security-dashboard-table')
page.within(find('aside')) do
expect(page).to have_content 'Vulnerabilities over time'
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment