[Interest] Problems customising Combobox popup with Qt 6.8.0

[EXT Mitch Curtis]

Ah, good catch! I've pushed a fix: https://codereview.qt-project.org/c/qt/qtdeclarative/+/604083

> -----Original Message-----
> From: Nuno Santos <nuno.santos at imaginando.pt>
> Sent: Wednesday, 13 November 2024 5:05 PM
> To: EXT Mitch Curtis <mitch.curtis at qt.io>
> Cc: interestqt-project. org <interest at qt-project.org>
> Subject: Re: [Interest] Problems customising Combobox popup with Qt 6.8.0
> 
> Mitch,
> 
> Thanks for your reply.
> 
> Inside QtCreator I’ve entered inside Combobox and seen the template for the
> Combobox customisation which is different from the documentation.
> 
> If I change this line:
> 
> implicitHeight: contentItem.implicitHeight
> 
> To
> 
> height: Math.min(contentItem.implicitHeight, control.Window.height -
> topMargin - bottomMargin)
> 
> It works as expected.
> 
> Now I don’t know if this is a documentation issue or a code issue.
> 
> Please advise and I will open the bug report.
> 
> Best regards,
> 
> Nuno
> 
> 
> 	On 13 Nov 2024, at 03:55, EXT Mitch Curtis <mitch.curtis at qt.io>
> wrote:
> 
> 	Hi,
> 
> 
> 
> 		-----Original Message-----
> 		From: Interest <interest-bounces at qt-project.org
> <mailto:interest-bounces at qt-project.org> > On Behalf Of Nuno Santos
> 		via Interest
> 		Sent: Wednesday, 13 November 2024 3:43 AM
> 		To: interestqt-project. org <interest at qt-project.org
> <mailto:interest at qt-project.org> >
> 		Subject: [Interest] Problems customising Combobox popup
> with Qt 6.8.0
> 
> 		Hi,
> 
> 		Since Qt 6.8.0, customising Combobox popup property makes
> the Combobox
> 		unscrollable when the list goes beyond the window height.
> However, if the
> 		popup is removed, scrolling works as expected.
> 
> 		Is there any particular detail on the documentation
> customisation code that
> 		hasn’t been updated yet?
> 
> 		ComboBox {
> 		       id: control
> 		       model: 100
> 		       popup: Popup {
> 		           y: control.height - 1
> 		           width: control.width
> 		           implicitHeight: contentItem.implicitHeight
> 		           padding: 1
> 		           contentItem: ListView {
> 		               clip: true
> 		               implicitHeight: contentHeight
> 		               model: control.popup.visible ? control.delegateModel :
> null
> 		               currentIndex: control.highlightedIndex
> 		               ScrollIndicator.vertical: ScrollIndicator { }
> 		           }
> 		           background: Rectangle {
> 		               border.color: "#21be2b"
> 		               radius: 2
> 		           }
> 		       }
> 		   }
> 
> 
> 		Project attached.
> 
> 
> 		Has anyone faced this problem?
> 
> 
> 
> 	I can reproduce this with 6.7.3 but not 6.6.3.
> 
> 	"git log v6.6.3..v6.7.3 --
> src/quicktemplates/qquickpopuppositioner.cpp" shows several changes and
> unfortunately a quick revert to check isn't on the cards here as it doesn't apply
> cleanly.
> 
> 	Can you please create a bug report?
> 
> 
> 
> 
> 		Thanks in advance!
> 
> 
> 		Regards,
> 
> 
> 		Nuno
>