[Bug]: Combobox heights and Text positioning not correct when Padding is applied to the DropDown Content

[Fungusware]

Describe the bug
Combobox heights and Text positioning not correct when Padding is applied to the DropDown Content

To Reproduce
Add a Krypton Combobox to a form and set the properties as shown

Expected behavior
Combobox should expand its height to fit the text and padding much like the Text box does.

Screenshots

Desktop (please complete the following information):

• OS: Windows 11
• Framework/.NET Version: [e.g. 6.0]
• Toolkit Version: 85.24.10.288

[giduac]

Hi @Smurf-IV,

The normal combobox does not have a padding property. The KCombo only receives this property through StateCommon.ComboBox.Content.

It looks like this is something that should not be there or has nothing todo with the inner combobox.

@Fungusware
Best to to leave this at it's default value.
What is it you are trying to accomplish ?

[Fungusware]

Thanks for the response. I would just like to make the Combo Box the same height as the textbox when the padding is added. Otherwise it looks a bit strange in the UI, See below. [image: image.png] Regards.

…

On Sat, Nov 2, 2024 at 3:46 PM giduac ***@***.***> wrote: Hi @Smurf-IV <https://github.com/Smurf-IV>, The normal combobox does not have a padding property. The KCombo only receives this property through StateCommon.ComboBox.Content. It looks like this something that should not be there or has nothing todo with the inner combobox. @Fungusware <https://github.com/Fungusware> Best to to leave this at it's default value. What is it you are trying to accomplish ? — Reply to this email directly, view it on GitHub <#1832 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEYHB5H64G6VBVNBT2IP6BLZ6R7NZAVCNFSM6AAAAABQ5PCCPKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJSHEYDSOJUGQ> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- -- John Anderson Fungusware http://fungusware.com http://www.xenserver-backup.com http://www.kojona.com

[Smurf-IV]

It looks like this is something that should not be there or has nothing todo with the inner combobox.

Yes it should, for curvature and other stylings to bring in line for Terminal style apps

@Fungusware Best to to leave this at it's default value. What is it you are trying to accomplish ?

Have you tried using V90 (Or alpha)

[Fungusware]

@Smurf-IV Is there a solid chance that v90 or higher would work? I'm quite locked to .net6.0 at the moment. I could upgrade assuming the changes are substantial and beneficial.

[Smurf-IV]

Just need it as a cross check, to see if the "fix" / regression has been done in V90.
Then that same "fix" can be retrofitted down into V85 LTS.
If not it a whole new investigation which could take longer.
Also, Can you add a simple demo, just in case it's a config / manifest issue ?

[Fungusware]

KryptonBugDemo.zip

https://github.com/Fungusware/KryptonBugDemo

No problem, and thanks for the fast assistance. The Toolkit is an integral part of my development life, so your attention is much appriciated.

Attached is a NET6 and NET8 example of the issue I am facing.

[giduac]

Hi @Fungusware,

I've removed the previous post and spent some more time on this.

Can you please test the class below and use this as combobox on the form.

• Make sure to set IntegralHeight to true, to show list items in full.
• Use the MinimumSize to force the control to use the desired size.
• OnSizeChanged() will vertically center the inner combobox.

Please let us know what you think it.

using System.ComponentModel;
using System.Diagnostics;

namespace KryptonBugDemoNet8
{
    public class CustomCombo : Krypton.Toolkit.KryptonComboBox
    {
        [DefaultValue(false)]
        [DesignerSerializationVisibility(DesignerSerializationVisibility.Visible)]
        public new bool IntegralHeight
        {
            get => base.IntegralHeight;
            set => base.IntegralHeight = value;
        }

        protected override void OnSizeChanged(EventArgs e)
        {
            base.OnSizeChanged(e);
            ComboBox.Top = (this.Height / 2) - (ComboBox.Height / 2);
        }
    }
}

[Fungusware]

I think its getting better, Applying rounding seemed to cause the text to jump back to the top though.

[giduac]

Nice so far.
This needs a bit more work. Also changing font size, padding, etc. need to be looked at.
I'll keep you posted.

[giduac]

Hi @Fungusware,

A bit further.

I'd please like you to clone and test this repo / branch:
https://github.com/giduac/Standard-Toolkit/tree/1832-V100-KryptonComboBox-does-not-respect-padding

You can build it from inside VStudio and run TestForm. In the StartScreen's lower right corner there's a button for the test.
Left propertygrid is for the KCombo
Middle for the inner combo
Right for the custom palette.

[giduac]

Hi @Fungusware,

A bit further.

I'd please like you to clone and test this repo / branch:
https://github.com/giduac/Standard-Toolkit/tree/1832-V100-KryptonComboBox-does-not-respect-padding

You can build it from inside VStudio and run TestForm. In the StartScreen's lower right corner there's a button for the test.
Left propertygrid is for the KCombo
Middle for the inner combo
Right for the custom palette.

[giduac]

Hi @Fungusware,

A bit further.

I'd please like you to clone and test this repo / branch:
https://github.com/giduac/Standard-Toolkit/tree/1832-V100-KryptonComboBox-does-not-respect-padding

You can build it from inside VStudio and run TestForm. In the StartScreen's lower right corner there's a button for the test.
Left propertygrid is for the KCombo
Middle for the inner combo
Right for the custom palette.

[Fungusware]

I'm having some issue getting the Solution to build. I'll ty and get back to you.

[giduac]

Hi @Fungusware,

The proposed changed has been merged and will be included in V85 next patch, V95 next patch and V100 dev branch.