-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature Request]: Controls should have "min design heights" be the same. #615
Comments
@Smurf-IV Is this possible? |
Probably, But I think having them all the same size would be a good start ! |
@MattH-Work Great find. |
https://docs.microsoft.com/en-us/dotnet/core/compatibility/winforms#change-description |
I'll make min heights to be 30, and that will cover both the above (i.e. 23 * 1.27 = 29.###) and when corners are used, and when a button spec is inserted. |
Note: Won;t fix for V85 as there are tooo many changes |
I have all my krypton controls set to 25, will I still be able to reduce the height lower than the 30 ( standard ) minimum you have proposed via the controls minimum height designer setting? |
Short answer: Long Answer:
|
This is not specific to Krypton controls. The corresponding WinForms controls (read: ComboBox, NumericUpDown, etc.) work the same way. Changing how the height of these controls is managed is likely to cause other problems if you're not careful (read more in my other comment). As to the alignment issue, I also like my controls to be aligned but the WinForms designer already has tools to help with this. Check the Format menu. I even have custom shortcuts to help me align controls quickly. |
Hi @Smurf-IV I might be missing something obvious but is this relevant? Hasn't Krypton Toolkit always used a different default font (which happens to be |
I'm aware of the alignment controls and use them frequently, however, even with all controls aligned to 'tops', the bottoms are up and down, it just doesn't look professional, hence why I opted ( for an OCD pleasing ) 25 height for all my controls, small enough to fit all required controls on the page, large enough to see |
Thanks for all the comments .. Keep them coming.
Proposal:
Thoughts ? |
Agreed. I like the idea of a standard height, uniform across all controls, but still needing the option to reduce / increase on a case by case basis using Minimum & Maximum Size control
Being selfish, as my button spec images are based on a standared icon size of 16x16, having the default standard control height set to 25 seems spot on
Not sure what you mean, don't they already align by tops, lefts, middles etc? |
Yes, some control's alignments properties IIRC need a check. |
Hi @Smurf-IV Note that even when Microsoft made the change you referred to above in .NET Core 3.0, they didn't increase the MinimumSize of the controls. They changed the font, which increased the default (not minimum) size of some controls. I'm not opposed to making changes to the default size, but I don't see a good reason to change the minimum size of all control. Maybe some controls need to have their size constrained (for example, a KTextBox with ButtonSpecs currently, doesn't shrink even if you use the smallest font possible, which is good), but this should be done on a control-by-control basis (where warranted), to avoid unintended side effects.
Is there a problem with that currently?
Default size? Maybe. Min size? I don't know. Why?
Maybe having a max size for button specs would help here? |
Hi @MattH-Work, @Smurf-IV, @giduac & @Ahmed-Abdelhameed Should I turn this into a discussion? |
See some additional thoughts here |
Is that not what you already have ? (AS you have already demonstrated via you use of MinSize !) Thanks for the Layout diagrams you did in the merge conversation, |
No, AS things get lost "Over there" |
1 - Agreed, Hence V90 only work.
|
1 - Ok, So Krypton accidentally did the same by also using Segoe UI 9pt ;-) |
It would be beneficial to allow 'input' controls (ComboBoxes, TextBoxes, NumericUpDown, DateTimePicker etc) to have their heights modifiable independently of the Font size.
Textbox height can change if set to multiline, whereas ComboBoxes (my main reason for this post) and other controls are linked to their font size
Default Heights
Textbox 23
NumericUpDown 22
Combox 21
DateTimePicker 21
It’s only a pixel here and there ( slight OCD triggering, 1st world problems and all! ), but its noticeable when controls don't line up on forms
I do note that the height increases when using ButtonSpec, or adding Rounding to the borders, so it feels the code for manipulating the heights exists in the background
The text was updated successfully, but these errors were encountered: