search

Krypton-Suite / Standard-Toolkit

An update to Component factory's krypton toolkit to support .NET Framework 4.6.2 - 4.8.1 to .NET 8 - 9
BSD 3-Clause "New" or "Revised" License
418 stars 63 forks source link

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

Open Fungusware opened 4 weeks ago

Fungusware commented 4 weeks ago

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 image

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

Screenshots image

Desktop (please complete the following information):

giduac commented 3 weeks ago

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 commented 3 weeks ago

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 https://github.com/Krypton-Suite/Standard-Toolkit/issues/1832#issuecomment-2452909944, 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 commented 3 weeks ago

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 commented 3 weeks ago

@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 commented 3 weeks ago

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 commented 3 weeks ago

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.