feat(completion): size the description column by need, not by half

The delegate split each row evenly between the value and its
description. That reads as fair but spends the width on the wrong
column: the values are short, the longest prefix being "attachment:",
while the descriptions are ordinary prose. Measured against the built-in
vocabulary, ten of fifteen descriptions elided at a 400px popup and the
longest, "directory below the Maildir root", needed a ~650px popup to
appear in full, all while the value's half of the row sat mostly empty.
The descriptions are the whole reason the popup teaches the query
language, so a half they cannot use is a half wasted.

Measure the value and lend the description the remainder, capped at 65%
of the row. Every built-in description now fits at ~500px instead of
~700px. The cap is what keeps a genuinely long value legible, and the
value's own reservation is capped in turn so that
"application/vnd.oasis.opendocument.text" elides itself rather than
claiming the row and silencing the column that explains what it is.

The test renders the real popup and reads the pixels back, since the
delegate is private to the .cpp and legibility is a painting question.
It asserts the painted width against the width the text needs unelided,
not merely that something was drawn: a blank-or-not check passes against
the very rule this replaces. 550px is chosen deliberately, being a width
where the new rule paints the longest description in full and an even
split cannot; against the old code it fails with "painted 261px of the
285px it needs".

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

1 files changed, 26 insertions, 7 deletions

diff --git a/src/querycompleter.cpp b/src/querycompleter.cpp
index 7e9dfd9..9848faf 100644
--- a/src/querycompleter.cpp
+++ b/src/querycompleter.cpp
@@ -88,10 +88,10 @@ int tokenEnd(const QString &text, int cursor)
 
 /// Draws the description greyed and right-aligned beside the value.
 ///
-/// The two are drawn into disjoint halves of the row rather than simply
-/// painted on top of each other: a long value ("application/vnd.oasis..."
-/// exceeds the popup width on its own) would otherwise run underneath the
-/// description and render both unreadable.
+/// The two are drawn into disjoint spans of the row rather than simply painted
+/// on top of each other: a long value ("application/vnd.oasis..." exceeds the
+/// popup width on its own) would otherwise run underneath the description and
+/// render both unreadable. The split is not even, see paint().
 class CompletionDelegate : public QStyledItemDelegate
 {
 public:
@@ -111,11 +111,30 @@ public:
         const int gap = 12;
         const int rightMargin = 6;
 
-        // The description never takes more than its share, so a long value
-        // keeps room to be legible and a long description gets elided too.
+        // Give the description what the value does not need, up to a ceiling.
+        //
+        // An even split reads as fair but spends width on the wrong column:
+        // the values here are short (the longest prefix is "attachment:") while
+        // the descriptions are ordinary prose, so at a 400px popup two thirds
+        // of them elided while the value half sat mostly empty. Measuring the
+        // value and lending the description the remainder clears every built-in
+        // description at ~500px instead of ~700px.
+        //
+        // The ceiling still matters: a long value ("application/vnd.oasis...")
+        // must keep enough room to stay legible rather than be squeezed to an
+        // ellipsis by a description that happens to be wordy.
         const int available = option.rect.width() - gap - rightMargin;
+        const QString value = index.data(Qt::DisplayRole).toString();
+        const int valueWidth = metrics.horizontalAdvance(value);
+        const int ceiling = (available * 65) / 100;
+
+        // A value too long to fit would otherwise claim the whole row and
+        // leave no description at all. Cap what it can reserve, so an
+        // "application/vnd.oasis..." elides itself rather than silencing the
+        // column that explains what it is.
+        const int reservedForValue = qMin(valueWidth, available - ceiling);
         int descriptionWidth = qMin(metrics.horizontalAdvance(description),
-                                    available / 2);
+                                    qMin(available - reservedForValue, ceiling));
         descriptionWidth = qMax(descriptionWidth, 0);
 
         // Let the base class draw the selection background and the value, but