Commit a7423067 authored by Thomas Randolph's avatar Thomas Randolph Committed by Phil Hughes

Handle when attention requests is turned off with better sidebar source layout

parent d853a87a
...@@ -113,13 +113,14 @@ export default { ...@@ -113,13 +113,14 @@ export default {
'gl-display-grid gl-align-items-center': showVerticalList, 'gl-display-grid gl-align-items-center': showVerticalList,
'gl-mb-3': index !== users.length - 1 && showVerticalList, 'gl-mb-3': index !== users.length - 1 && showVerticalList,
}" }"
class="assignee-attention-grid" class="assignee-grid"
> >
<assignee-avatar-link <assignee-avatar-link
:user="user" :user="user"
:issuable-type="issuableType" :issuable-type="issuableType"
:tooltip-has-name="!showVerticalList" :tooltip-has-name="!showVerticalList"
class="gl-grid-column-2 gl-grid-row-1 gl-word-break-word" class="gl-word-break-word"
data-css-area="user"
> >
<div <div
v-if="showVerticalList" v-if="showVerticalList"
...@@ -134,7 +135,8 @@ export default { ...@@ -134,7 +135,8 @@ export default {
v-if="showVerticalList" v-if="showVerticalList"
:user="user" :user="user"
type="assignee" type="assignee"
class="gl-grid-column-1 gl-grid-row-1 gl-mr-2" class="gl-mr-2"
data-css-area="attention"
@toggle-attention-requested="toggleAttentionRequested" @toggle-attention-requested="toggleAttentionRequested"
/> />
</div> </div>
......
...@@ -94,15 +94,19 @@ export default { ...@@ -94,15 +94,19 @@ export default {
<div <div
v-for="(user, index) in users" v-for="(user, index) in users"
:key="user.id" :key="user.id"
:class="{ 'gl-mb-3': index !== users.length - 1 }" :class="{
class="gl-display-grid gl-align-items-center reviewer-attention-grid" 'gl-mb-3': index !== users.length - 1,
'attention-requests': glFeatures.mrAttentionRequests,
}"
class="gl-display-grid gl-align-items-center reviewer-grid"
data-testid="reviewer" data-testid="reviewer"
> >
<reviewer-avatar-link <reviewer-avatar-link
:user="user" :user="user"
:root-path="rootPath" :root-path="rootPath"
:issuable-type="issuableType" :issuable-type="issuableType"
class="gl-grid-column-2 gl-grid-row-1 gl-word-break-word gl-mr-2" class="gl-word-break-word gl-mr-2"
data-css-area="user"
> >
<div class="gl-ml-3 gl-line-height-normal gl-display-grid"> <div class="gl-ml-3 gl-line-height-normal gl-display-grid">
<span>{{ user.name }}</span> <span>{{ user.name }}</span>
...@@ -113,7 +117,8 @@ export default { ...@@ -113,7 +117,8 @@ export default {
v-if="glFeatures.mrAttentionRequests" v-if="glFeatures.mrAttentionRequests"
:user="user" :user="user"
type="reviewer" type="reviewer"
class="gl-grid-column-1 gl-grid-row-1 gl-mr-2" class="gl-mr-2"
data-css-area="attention"
@toggle-attention-requested="toggleAttentionRequested" @toggle-attention-requested="toggleAttentionRequested"
/> />
<gl-icon <gl-icon
......
...@@ -227,22 +227,28 @@ ...@@ -227,22 +227,28 @@
margin-right: -$gl-spacing-scale-2; margin-right: -$gl-spacing-scale-2;
} }
.reviewer-attention-grid, .assignee-grid {
.assignee-attention-grid { grid-template-areas: ' attention user';
grid-template-columns: min-content 1fr min-content; grid-template-columns: min-content 1fr;
} }
/* TODO: These are non-standardized classes, and should be moved into gitlab-ui .reviewer-grid {
Please see: https://gitlab.com/gitlab-org/gitlab-ui/-/issues/1780 grid-template-areas: ' user approval rerequest';
*/ grid-template-columns: 1fr min-content min-content;
.gl-grid-column-1 {
grid-column: 1;
}
.gl-grid-row-1 { &.attention-requests {
grid-row: 1; grid-template-areas: ' attention user approval';
grid-template-columns: min-content 1fr min-content;
}
} }
.gl-grid-column-2 { .assignee-grid,
grid-column: 2; .reviewer-grid {
[data-css-area='attention'] {
grid-area: attention;
}
[data-css-area='user'] {
grid-area: user;
}
} }
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