commit 055fddbca0efb85e031fbabdf88211fa28bab27e
author Caolán McNamara <caolanm@redhat.com> Wed Feb 10 19:52:52 2021 +0000
committer Adolfo Jayme Barrientos <fitojb@ubuntu.com> Thu Feb 11 17:41:33 2021 +0100
tree 4b3d2ccb6554b6cf7e49405ac9b79c5254fe34bb
parent 08b6d409b2b5228c14e8b94eddbb8987ec6e9c78

tdf#140006 don't limit dropdown width to 300pixels

let it use the full width of the cell. Keep that 300 pixel limit
if we are expanding the widget width beyond its size request.

Change-Id: Ifeaa9ee60e0d5649de71c960ae83270fb1afb2e7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/110682
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>

sc/source/ui/view/gridwin.cxx

diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx
index 34d3ed0..fa8b2a4 100644
--- a/sc/source/ui/view/gridwin.cxx
+++ b/sc/source/ui/view/gridwin.cxx
@@ -959,10 +959,9 @@ void ScGridWindow::ShowFilterMenu(const tools::Rectangle& rCellRect, bool bLayou
    auto nHeight = rFilterBox.get_height_rows(nEntryCount);
    rFilterBox.set_size_request(-1, nHeight);
    Size aSize(rFilterBox.get_preferred_size());
    if (aSize.Width() < nSizeX)
        aSize.setWidth(nSizeX);
    if (aSize.Width() > 300)
        aSize.setWidth(300);     // do not over do it (Pixel)
    auto nMaxToExpandTo = std::min(nSizeX, static_cast<decltype(nSizeX)>(300));     // do not over do it (Pixel)
    if (aSize.Width() < nMaxToExpandTo)
        aSize.setWidth(nMaxToExpandTo);

    aSize.AdjustWidth(4); // add a little margin
    nSizeX += 4;