Skip to content

Commit b9ab40c

Browse files
position for multiple fields and fix tooltip css
1 parent d31f6c0 commit b9ab40c

2 files changed

Lines changed: 23 additions & 38 deletions

File tree

src/visualBuilder/components/FieldToolbar.tsx

Lines changed: 21 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -309,6 +309,18 @@ function FieldToolbarComponent(
309309
fetchFieldSchema();
310310
}, [fieldMetadata]);
311311

312+
const multipleFieldToolbarButtonClasses = classNames(
313+
"visual-builder__button visual-builder__button--secondary",
314+
visualBuilderStyles()["visual-builder__button"],
315+
visualBuilderStyles()["visual-builder__button--secondary"],
316+
visualBuilderStyles()["visual-builder__tooltip"],
317+
{
318+
"visual-builder__tooltip--bottom": invertTooltipPosition,
319+
[visualBuilderStyles()["visual-builder__tooltip--bottom"]]:
320+
invertTooltipPosition,
321+
}
322+
);
323+
312324
return (
313325
<div
314326
className={classNames(
@@ -348,18 +360,9 @@ function FieldToolbarComponent(
348360
<>
349361
<button
350362
data-testid="visual-builder__focused-toolbar__multiple-field-toolbar__move-left-button"
351-
className={classNames(
352-
`visual-builder__button visual-builder__button--secondary`,
353-
visualBuilderStyles()[
354-
"visual-builder__button"
355-
],
356-
visualBuilderStyles()[
357-
"visual-builder__button--secondary"
358-
],
359-
visualBuilderStyles()[
360-
"visual-builder__tooltip"
361-
]
362-
)}
363+
className={
364+
multipleFieldToolbarButtonClasses
365+
}
363366
data-tooltip={
364367
direction.value === "vertical"
365368
? "Move up"
@@ -389,18 +392,9 @@ function FieldToolbarComponent(
389392

390393
<button
391394
data-testid="visual-builder__focused-toolbar__multiple-field-toolbar__move-right-button"
392-
className={classNames(
393-
`visual-builder__button visual-builder__button--secondary`,
394-
visualBuilderStyles()[
395-
"visual-builder__button"
396-
],
397-
visualBuilderStyles()[
398-
"visual-builder__button--secondary"
399-
],
400-
visualBuilderStyles()[
401-
"visual-builder__tooltip"
402-
]
403-
)}
395+
className={
396+
multipleFieldToolbarButtonClasses
397+
}
404398
data-tooltip={
405399
direction.value === "vertical"
406400
? "Move down"
@@ -434,18 +428,9 @@ function FieldToolbarComponent(
434428

435429
<button
436430
data-testid="visual-builder__focused-toolbar__multiple-field-toolbar__delete-button"
437-
className={classNames(
438-
"visual-builder__button visual-builder__button--secondary",
439-
visualBuilderStyles()[
440-
"visual-builder__button"
441-
],
442-
visualBuilderStyles()[
443-
"visual-builder__button--secondary"
444-
],
445-
visualBuilderStyles()[
446-
"visual-builder__tooltip"
447-
]
448-
)}
431+
className={
432+
multipleFieldToolbarButtonClasses
433+
}
449434
data-tooltip={"Delete"}
450435
onClick={(e) => {
451436
e.preventDefault();

src/visualBuilder/visualBuilder.style.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ const tooltipBaseStyle = `
2626
&:after {
2727
content: "";
2828
position: absolute;
29-
bottom: 25px;
29+
bottom: 28px;
3030
/* the arrow */
3131
border: 10px solid #000;
3232
border-color: #767676 transparent transparent transparent;
@@ -442,7 +442,7 @@ export function visualBuilderStyles() {
442442
bottom: -66px;
443443
}
444444
&:after {
445-
bottom: -4px;
445+
bottom: -6px;
446446
transform: rotate(180deg);
447447
}
448448
`,

0 commit comments

Comments
 (0)