| Summary: | Context menu in Math text window doesn't stay if not enough space below cursor | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Mike Kaganski <mikekaganski> |
| Component: | Formula Editor | Assignee: | Caolán McNamara <caolan.mcnamara> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | 79045_79045, caolan.mcnamara |
| Priority: | medium | Keywords: | bibisected, bisected, regression |
| Version: | 7.2.0.4 release | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | target:7.4.0 target:7.3.0.0.beta2 target:7.2.5 | ||
| Crash report or crash signature: | Regression By: | ||
| Bug Depends on: | |||
| Bug Blocks: | 86349 | ||
| Attachments: | Screen cast of bringing context menu in two positions | ||
|
Description
Mike Kaganski
2021-11-30 07:53:26 UTC
confirm in Version: 7.4.0.0.alpha0+ (x64) / LibreOffice Community Build ID: d272bf4f8f3a083b8c37f7b10f89cd2bf76b7d21 CPU threads: 4; OS: Windows 6.1 Service Pack 1 Build 7601; UI render: default; VCL: win Locale: ru-RU (ru_RU); UI: en-US Calc: CL and in 7.2.0.4 but not in 7.1.7.2 so it's a regression I bisected this one in win64-7.2 bisect repo and got sha: 4c8e2b47cb81a8b582d71db927682753c6d12ad9 https://git.libreoffice.org/core/commit/4c8e2b47cb81a8b582d71db927682753c6d12ad9 https://gerrit.libreoffice.org/c/core/+/115293 Added to CC: Caolán McNamara I think how this works in vcl is that the popup has to be under the mouse pos, so its probably a positioning adjustment of something like move it -1 if using a fallback position to position the popup under the mouse. Caolán McNamara committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/4321fb516f4a65b047b182c4cb6f9c9a232efd85 Resolves: tdf#145959 menu pos should be relative to SmCmdBoxWindow It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. that seems to be the issue, fixed in trunk and backports to 7-3 and 7-2 in gerrit Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-7-3": https://git.libreoffice.org/core/commit/ea1afe06aaa54385c821267433d052a57131bc4c Resolves: tdf#145959 menu pos should be relative to SmCmdBoxWindow It will be available in 7.3.0.0.beta2. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-7-2": https://git.libreoffice.org/core/commit/0d4568f4b082b0f3cce97b4f18c2dc1ff60aeb85 Resolves: tdf#145959 menu pos should be relative to SmCmdBoxWindow It will be available in 7.2.4. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Verified in Version: 7.4.0.0.alpha0+ (x64) / LibreOffice Community Build ID: 2934472ab888ebfe64a153984af2902fac63a7a0 CPU threads: 4; OS: Windows 6.1 Service Pack 1 Build 7601; UI render: default; VCL: win Locale: ru-RU (ru_RU); UI: en-US Calc: CL Thank you, Caolán. 7.2.4 was a hotfix release, updating target in status-whiteboard |