• 
      

    Fix scrolling in the text editor in the comment dialog.

    Review Request #13929 — Created June 4, 2024 and submitted — Latest diff uploaded

    Information

    Review Board
    release-7.x

    Reviewers

    The fix to prevent accidentally scrolling away from the comment dialog
    broke scrolling within the text editor. This change adds some smarts so
    we figure out if we're trying to scroll in the text editor and the text
    editor is capable of scrolling, we'll allow it, and if not, we'll
    continue to swallow the event.

    • Verified that the scroll wheel did not do anything when moused over
      parts of the comment dialog other than the text editor.
    • Verified that the scroll wheel did not do anything when moused over an
      empty text field, in both markdown and plain text modes.
    • Verified that the scroll wheel did not do anything when moused over
      text that did not overflow the text editor in markdown mode.
    • Verified that the scroll wheel scrolled the text editor when working
      with long comments in both markdown and plan text modes.
    • Ran js-tests.

    Commits

    Files