Skip to content

Commit 8bb820e

Browse files
authored
Merge pull request #663 from RedisInsight/feature/bugfix
#RI-2906 - add tooltip for clear btn
2 parents aca28d5 + 3d14627 commit 8bb820e

File tree

2 files changed

+17
-10
lines changed

2 files changed

+17
-10
lines changed

redisinsight/ui/src/pages/browser/components/key-details-add-items/add-stream-entity/AddStreamEntries.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,10 @@ const AddStreamEntries = (props: Props) => {
4545
return
4646
}
4747

48-
if (!lastEntry.id) return
48+
if (!lastEntry?.id) {
49+
setEntryIdError('')
50+
return
51+
}
4952

5053
if (entryID === '*') {
5154
setEntryIdError('')

redisinsight/ui/src/pages/slowLog/components/Actions/Actions.tsx

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,6 @@ const Actions = (props: Props) => {
120120
/>
121121
</EuiFlexItem>
122122
<EuiFlexItem>
123-
124123
<EuiPopover
125124
ownFocus
126125
anchorPosition="downRight"
@@ -153,14 +152,19 @@ const Actions = (props: Props) => {
153152
closePopover={closePopoverClear}
154153
panelPaddingSize="m"
155154
button={(
156-
<EuiButtonIcon
157-
iconType="eraser"
158-
className={styles.icon}
159-
color="primary"
160-
aria-label="Clear Slow Log"
161-
onClick={() => showClearPopover()}
162-
data-testid="clear-btn"
163-
/>
155+
<EuiToolTip
156+
position="left"
157+
anchorClassName={styles.icon}
158+
content="Clear Slow Log"
159+
>
160+
<EuiButtonIcon
161+
iconType="eraser"
162+
color="primary"
163+
aria-label="Clear Slow Log"
164+
onClick={() => showClearPopover()}
165+
data-testid="clear-btn"
166+
/>
167+
</EuiToolTip>
164168
)}
165169
>
166170
{ToolTipContent}

0 commit comments

Comments
 (0)