Skip to content

Commit

Permalink
fix: Add back mapping hint when parameter is focused (#4634)
Browse files Browse the repository at this point in the history
  • Loading branch information
mutdmour authored Nov 17, 2022
1 parent 376a0d0 commit b35172e
Showing 1 changed file with 15 additions and 1 deletion.
16 changes: 15 additions & 1 deletion packages/editor-ui/src/components/RunDataTable.vue
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@
:class="{
[$style.header]: true,
[$style.draggableHeader]: mappingEnabled,
[$style.activeHeader]: i === activeColumn && mappingEnabled,
[$style.activeHeader]: (i === activeColumn || forceShowGrip) && mappingEnabled,
[$style.draggingHeader]: isDragging,
}"
>
Expand Down Expand Up @@ -202,6 +202,7 @@ export default mixins(externalHooks).extend({
data() {
return {
activeColumn: -1,
forceShowGrip: false,
draggedColumn: false,
draggingPath: null as null | string,
hoveringPath: null as null | string,
Expand Down Expand Up @@ -234,6 +235,9 @@ export default mixins(externalHooks).extend({
tableData(): ITableData {
return this.convertToTable(this.inputData);
},
focusedMappableInput(): string {
return this.ndvStore.focusedMappableInput;
},
},
methods: {
shorten,
Expand Down Expand Up @@ -494,6 +498,16 @@ export default mixins(externalHooks).extend({
this.$emit('displayModeChange', 'json');
},
},
watch: {
focusedMappableInput(curr: boolean) {
setTimeout(
() => {
this.forceShowGrip = !!this.focusedMappableInput;
},
curr ? 300 : 150,
);
},
},
});
</script>

Expand Down

0 comments on commit b35172e

Please sign in to comment.