Project

General

Profile

Actions
Copy link

Bug #13075

closed
SE RM

Large popup menus go off screen at the top

Bug #13075: Large popup menus go off screen at the top

Added by Stephan Ehlers almost 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Romain Mardulyn
Target version:
4.11.4
Start date:
10/02/2024
Due date:
% Done:

100%

Estimated time:

Description

If a popup menu (e.g. a context menu on some UI element) is large enough but not larger than the available total space (usually the window height), it can be placed in such a way that its top goes off screen.

"Large enough" in this case means that the menu is too large to be displayed "hanging" off the mouse cursor without cutting off its bottom, yet also too large to be shown above the current cursor position. The current algorithm only checks for the former case.

I created a PR on github (https://github.com/emweb/wt/pull/220) to fix this issue; it is currently based on 4.10.4, but could easily be rebased to 4.11.0.

Related issues 1 (0 open — 1 closed)

Related to Bug #13798: Submenu does not open if browser window is too narrowClosedRomain Mardulyn06/24/2025

Actions

SE Updated by Stephan Ehlers almost 2 years ago Actions
Copy link
 #1

To add some numbers to the example:
Imagine a context menu that is 51% as tall as the screen, and you want to open it exactly at the center of the screen. It will be displayed above the cursor, since there is not enough space below it. But then the top 1% of the menu will be cut off.

MV Updated by Matthias Van Ceulebroeck almost 2 years ago Actions
Copy link
 #2

  • Target version set to 4.11.1

Thank you for the submission Stephan.
I will soon review the implementation and test it, to see if I can find any edge cases to it.

MV Updated by Matthias Van Ceulebroeck almost 2 years ago Actions
Copy link
 #3

  • Target version changed from 4.11.1 to 4.11.3

MV Updated by Matthias Van Ceulebroeck over 1 year ago Actions
Copy link
 #4

  • Target version changed from 4.11.3 to 4.11.4

MV Updated by Matthias Van Ceulebroeck over 1 year ago Actions
Copy link
 #5

  • Status changed from New to InProgress
  • Assignee changed from Stephan Ehlers to Romain Mardulyn

MV Updated by Matthias Van Ceulebroeck over 1 year ago Actions
Copy link
 #6

  • Status changed from InProgress to Review
  • Assignee changed from Romain Mardulyn to Matthias Van Ceulebroeck

MV Updated by Matthias Van Ceulebroeck over 1 year ago Actions
Copy link
 #7

  • Status changed from Review to Implemented @Emweb
  • Assignee changed from Matthias Van Ceulebroeck to Romain Mardulyn
  • % Done changed from 80 to 100

MV Updated by Matthias Van Ceulebroeck over 1 year ago Actions
Copy link
 #8

  • Status changed from Implemented @Emweb to Implemented @Test

MV Updated by Matthias Van Ceulebroeck over 1 year ago Actions
Copy link
 #9

  • Status changed from Implemented @Test to Closed

MV Updated by Matthias Van Ceulebroeck about 1 year ago Actions
Copy link
 #10

  • Related to Bug #13798: Submenu does not open if browser window is too narrow added
Actions
Copy link

Also available in: PDF Atom