Skip to content

Commit

Permalink
UI: Remove ok confirmation for create/edit operation (#1304)
Browse files Browse the repository at this point in the history 
* remove ok confirmation

Signed-off-by: craman <[email protected]>

* update group role page

Signed-off-by: craman <[email protected]>

* fix-lint

Signed-off-by: craman <[email protected]>

* address PR comments

Signed-off-by: craman <[email protected]>

Co-authored-by: craman <[email protected]>
  • Loading branch information
@chandrasekhar1996
chandrasekhar1996 and craman authored Mar 3, 2021
1 parent 78d9e9f commit c41ed9b
Show file tree
Hide file tree
Showing 41 changed files with 930 additions and 561 deletions.
104 changes: 56 additions & 48 deletions ui/src/__tests__/components/member/__snapshots__/MemberList.test.js.snap
View file
Original file line number Diff line number Diff line change
Expand Up @@ -147,30 +147,30 @@ exports[`MemberList should render 1`] = `
word-break: break-all;
}
.emotion-22 {
.emotion-24 {
background-color: #3570f40D;
text-align: left;
padding: 5px 0 5px 15px;
vertical-align: middle;
word-break: break-all;
}
.emotion-30 {
.emotion-32 {
background-color: #3570f40D;
text-align: center;
padding: 5px 0 5px 15px;
vertical-align: middle;
word-break: break-all;
}
.emotion-33 {
.emotion-37 {
text-align: left;
padding: 5px 0 5px 15px;
vertical-align: middle;
word-break: break-all;
}
.emotion-41 {
.emotion-45 {
text-align: center;
padding: 5px 0 5px 15px;
vertical-align: middle;
Expand Down Expand Up @@ -251,32 +251,33 @@ exports[`MemberList should render 1`] = `
</thead>
<tbody>
<tr
class="emotion-22 emotion-23"
data-testid="member-row"
>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
user1
</td>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
/>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
N/A
</td>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
N/A
</td>
<td
class="emotion-30 emotion-23"
class="emotion-32 emotion-25"
color="#3570f40D"
>
<span>
Expand All @@ -301,32 +302,33 @@ exports[`MemberList should render 1`] = `
</td>
</tr>
<tr
class="emotion-22 emotion-23"
data-testid="member-row"
>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
user4
</td>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
/>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
N/A
</td>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
N/A
</td>
<td
class="emotion-41 emotion-23"
class="emotion-45 emotion-25"
color=""
>
<span>
Expand Down Expand Up @@ -411,32 +413,33 @@ exports[`MemberList should render 1`] = `
</thead>
<tbody>
<tr
class="emotion-22 emotion-23"
data-testid="member-row"
>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
user2
</td>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
/>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
N/A
</td>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
N/A
</td>
<td
class="emotion-30 emotion-23"
class="emotion-32 emotion-25"
color="#3570f40D"
>
<span>
Expand All @@ -461,32 +464,33 @@ exports[`MemberList should render 1`] = `
</td>
</tr>
<tr
class="emotion-22 emotion-23"
data-testid="member-row"
>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
user3
</td>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
/>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
N/A
</td>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
N/A
</td>
<td
class="emotion-41 emotion-23"
class="emotion-45 emotion-25"
color=""
>
<span>
Expand Down Expand Up @@ -662,30 +666,30 @@ exports[`MemberList should render delegated role 1`] = `
word-break: break-all;
}
.emotion-22 {
.emotion-24 {
background-color: #3570f40D;
text-align: left;
padding: 5px 0 5px 15px;
vertical-align: middle;
word-break: break-all;
}
.emotion-30 {
.emotion-32 {
background-color: #3570f40D;
text-align: center;
padding: 5px 0 5px 15px;
vertical-align: middle;
word-break: break-all;
}
.emotion-33 {
.emotion-37 {
text-align: left;
padding: 5px 0 5px 15px;
vertical-align: middle;
word-break: break-all;
}
.emotion-41 {
.emotion-45 {
text-align: center;
padding: 5px 0 5px 15px;
vertical-align: middle;
Expand Down Expand Up @@ -766,32 +770,33 @@ exports[`MemberList should render delegated role 1`] = `
</thead>
<tbody>
<tr
class="emotion-22 emotion-23"
data-testid="member-row"
>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
user1
</td>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
/>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
N/A
</td>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
N/A
</td>
<td
class="emotion-30 emotion-23"
class="emotion-32 emotion-25"
color="#3570f40D"
>
<span>
Expand All @@ -816,32 +821,33 @@ exports[`MemberList should render delegated role 1`] = `
</td>
</tr>
<tr
class="emotion-22 emotion-23"
data-testid="member-row"
>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
user2
</td>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
/>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
N/A
</td>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
N/A
</td>
<td
class="emotion-41 emotion-23"
class="emotion-45 emotion-25"
color=""
>
<span>
Expand All @@ -866,32 +872,33 @@ exports[`MemberList should render delegated role 1`] = `
</td>
</tr>
<tr
class="emotion-22 emotion-23"
data-testid="member-row"
>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
user3
</td>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
/>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
N/A
</td>
<td
class="emotion-22 emotion-23"
class="emotion-24 emotion-25"
color="#3570f40D"
>
N/A
</td>
<td
class="emotion-30 emotion-23"
class="emotion-32 emotion-25"
color="#3570f40D"
>
<span>
Expand All @@ -916,32 +923,33 @@ exports[`MemberList should render delegated role 1`] = `
</td>
</tr>
<tr
class="emotion-22 emotion-23"
data-testid="member-row"
>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
user4
</td>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
/>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
N/A
</td>
<td
class="emotion-33 emotion-23"
class="emotion-37 emotion-25"
color=""
>
N/A
</td>
<td
class="emotion-41 emotion-23"
class="emotion-45 emotion-25"
color=""
>
<span>
Expand Down
Loading

0 comments on commit c41ed9b

Please sign in to comment.