Fix settings scroll problem

[GretaD]

fixes #2967

before - check fixes

after

[codecov]

Codecov Report

Merging #3026 (10bf92b) into master (e79296b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3026   +/-   ##
=========================================
  Coverage     29.18%   29.18%           
  Complexity      116      116           
=========================================
  Files           155      155           
  Lines          5561     5561           
  Branches        822      822           
=========================================
  Hits           1623     1623           
  Misses         3938     3938           

Flag	Coverage Δ	Complexity Δ
javascript	23.85% <ø> (ø)	0.00 <ø> (ø)
php	94.29% <ø> (ø)	116.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e79296b...10bf92b. Read the comment docs.

[raimund-schluessler]

I think overflow: visible is there to prevent issues with the multiselects for timezone and duration. overflow: auto makes it difficult to select a value and leads to a scrollbar in y-direction for some locals. At least this was the status of the discussion in #2516.

[GretaD]

I think overflow: visible is there to prevent issues with the multiselects for timezone and duration. overflow: auto makes it difficult to select a value and leads to a scrollbar in y-direction for some locals. At least this was the status of the discussion in #2516.

ohh i see, thank you for pointing it out, i was trying to find smth if it was done on purpose but failed. So I guess, would be better to put the whole settings into a modal and this way we fix the "crowed" problem and this bug.

[GretaD]

We decided on Groupware channel to move settings into a modal, so it will be fixed there.