Optimize: Rawdata report settings tab is not scrollable

[venetrius]

Environment (Required on creation)

Optimize 3.15

Description (Required on creation; please attach any relevant screenshots, stacktraces, log files, etc. to the ticket)

Steps to reproduce (Required on creation)

• Create a Process Report of type raw data (from a process definition that has many variables)

screenshot

• activate “update preview automatically"
• next to visualization press the “gear”,
• adjust the window size in a way that settings popup fits in the view perfectly.

screenshot

• open the VARIABLES Tab, there is no scrollbar

screenshot

Observed Behavior (Required on creation)

The settings tab does not fit in the screen, but no scroll bar is present

Expected behavior (Required on creation)

If the settings popover does not fit in screen, then the scroll bar is always present

Root Cause (Required on prioritization)

The tab is rendered by the Popover component.
calculateDialogStyle is the function that sets scrollable.
calculateDialogStyle is only called on handleResize. This happens if the size of the browser window is changed. This is not triggered when the Popover component size has changed

Solution Ideas

Resize event can be triggered via window.dispatchEvent(new Event('resize')); from Configuration which renderes Popover

Hints

In Popover calculateDialogStyle it need to be checked that all size are updated accordingly to the current DOM.

Conditionals that sets scrollable

    if (bottomSpace < contentHeightWithPadding && topSpace > bottomSpace) {
      const scrollable = contentHeightWithPadding > topSpace;
      setScrollable(scrollable);
      dialogStyles.height = (scrollable ? topSpace : contentHeightWithPadding) + 'px';
      newAlignment = newAlignment.replace('bottom', 'top') as PopoverAlignment;
    }

Links

SUPPORT-25801

Breakdown

Pull Requests

Preview Give feedback

1. https://github.com/camunda/camunda-optimize/pull/15429

Dev2QA handover

• Does this ticket need a QA test and the testing goals are not clear from the description? Add a Dev2QA handover comment