Skip to content

Commit 1ab2b61

Browse files
giduacgiduac
authored
1376-V100-Extending-KryptonTaskDialog (#2497)
* 1376-V100-Extending-KryptonTaskDialog - Adds ProgressBar elements - Changes OnSizeChanged to a virtual method * 1376-V100-Extending-KryptonTaskDialog - Adds ProgressBar elements - Changes OnSizeChanged to a virtual method
1 parent eb34125 commit 1ab2b61

13 files changed

+502
-149
lines changed

‎Source/Krypton Components/Krypton.Toolkit/Controls Toolkit/KryptonTaskDialog/Elements/KryptonTaskDialogElementBase.cs‎

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,16 @@ public KryptonTaskDialogElementBase(KryptonTaskDialogDefaults taskDialogDefaults
7575
#endregion
7676

7777
#region Protected
78-
protected void OnSizeChanged()
78+
/// <summary>
79+
/// Virtual method that invokes the SizeChanged event and notifies subscribers.
80+
/// </summary>
81+
/// <param name="performLayout">If a layout needs to be performed regardless of the element's visible state.</param>
82+
protected virtual void OnSizeChanged(bool performLayout = false)
7983
{
80-
SizeChanged?.Invoke();
84+
if (!performLayout)
85+
{
86+
SizeChanged?.Invoke();
87+
}
8188
}
8289
#endregion
8390

@@ -215,7 +222,7 @@ private void OnBackColorsChanged()
215222
}
216223
}
217224

218-
private void OnVisibleChanged()
225+
protected virtual void OnVisibleChanged()
219226
{
220227
VisibleChanged?.Invoke();
221228
}

‎Source/Krypton Components/Krypton.Toolkit/Controls Toolkit/KryptonTaskDialog/Elements/KryptonTaskDialogElementCheckBox.cs‎

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -27,35 +27,31 @@ public KryptonTaskDialogElementCheckBox(KryptonTaskDialogDefaults taskDialogDefa
2727

2828
_checkBox = new();
2929
_checkBox.AutoSize = true;
30-
_checkBox.Padding = new Padding(5, 0, 0, 0);
31-
_checkBox.Margin = _nullMargin;
30+
_checkBox.Padding = new Padding(Defaults.ComponentSpace, 0, 0, 0);
31+
_checkBox.Margin = Defaults.NullMargin;
3232
_checkBox.CheckState = CheckState.Unchecked;
3333

3434
_tlp.Controls.Add(_checkBox, 0, 0);
3535

3636
LayoutDirty = true;
3737
}
3838

39-
#region Private
40-
#endregion
41-
private void OnSizeChanged(bool performLayout = false)
39+
#region Protected/Internal
40+
/// <inheritdoc/>
41+
protected override void OnSizeChanged(bool performLayout = false)
4242
{
4343
// Updates / changes are deferred if the element is not visible or until PerformLayout is called
4444
if (LayoutDirty && (Visible || performLayout))
4545
{
4646
Panel.Height = _checkBox.Height + Defaults.PanelTop + Defaults.PanelBottom;
4747

48-
if (!performLayout)
49-
{
50-
base.OnSizeChanged();
51-
}
48+
base.OnSizeChanged(performLayout);
5249

5350
LayoutDirty = false;
54-
5551
}
5652
}
5753

58-
#region Protected/Internal
54+
/// <inheritdoc/>
5955
protected override void OnPalettePaint(object? sender, PaletteLayoutEventArgs e)
6056
{
6157
base.OnPalettePaint(sender, e);
@@ -69,6 +65,7 @@ protected override void OnPalettePaint(object? sender, PaletteLayoutEventArgs e)
6965
}
7066
}
7167

68+
/// <inheritdoc/>
7269
internal override void PerformLayout()
7370
{
7471
base.PerformLayout();

‎Source/Krypton Components/Krypton.Toolkit/Controls Toolkit/KryptonTaskDialog/Elements/KryptonTaskDialogElementComboBox.cs‎

Lines changed: 18 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,23 @@ public KryptonTaskDialogElementComboBox(KryptonTaskDialogDefaults taskDialogDefa
6161
#endregion
6262

6363
#region Override
64+
/// <inheritdoc/>
65+
protected override void OnSizeChanged(bool performLayout = false)
66+
{
67+
if (LayoutDirty && (Visible || performLayout))
68+
{
69+
int height = ShowDescription
70+
? _description.Height + _description.Margin.Bottom :
71+
0;
72+
73+
Panel.Height = Defaults.PanelTop + Defaults.PanelBottom + _comboBox.Height + height;
74+
75+
base.OnSizeChanged(performLayout);
76+
77+
LayoutDirty = false;
78+
}
79+
}
80+
6481
/// <inheritdoc/>
6582
internal override void PerformLayout()
6683
{
@@ -182,7 +199,7 @@ private void SetupPanel()
182199
{
183200
// the description that goes with the combobox
184201
_description.AutoSize = true;
185-
_description.Padding = _nullPadding;
202+
_description.Padding = Defaults.NullPadding;
186203
_description.Margin = new(0, 0, 0, Defaults.ComponentSpace);
187204

188205
_comboBox.AutoSize = true;
@@ -206,25 +223,6 @@ private void OnComboBoxSizeChanged(object? sender, EventArgs e)
206223
OnSizeChanged();
207224
}
208225

209-
private void OnSizeChanged(bool performLayout = false)
210-
{
211-
if (LayoutDirty && (Visible || performLayout))
212-
{
213-
int height = ShowDescription
214-
? _description.Height + _description.Margin.Bottom :
215-
0;
216-
217-
Panel.Height = Defaults.PanelTop + Defaults.PanelBottom + _comboBox.Height + height;
218-
219-
if (!performLayout)
220-
{
221-
base.OnSizeChanged();
222-
}
223-
224-
LayoutDirty = false;
225-
}
226-
}
227-
228226
private void OnSelectedIndexChanged(object? sender, EventArgs e)
229227
{
230228
// Only fire if the dialog is visible

‎Source/Krypton Components/Krypton.Toolkit/Controls Toolkit/KryptonTaskDialog/Elements/KryptonTaskDialogElementCommonButtons.cs‎

Lines changed: 17 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,23 @@ private void UpdateButtonSize()
8080
}
8181

8282
#region Protected/Internal
83+
/// <inheritdoc/>
84+
protected override void OnSizeChanged(bool performLayout = false)
85+
{
86+
// Updates / changes are deferred if the element is not visible or until PerformLayout is called
87+
if (LayoutDirty && (Visible || performLayout))
88+
{
89+
// Button size is set through UpdateButtonSize() called from the constructor and OnPalettePaint.
90+
Panel.Height = _btnOk.Height + Defaults.PanelTop + Defaults.PanelBottom;
91+
92+
// Done
93+
LayoutDirty = false;
94+
95+
// Tell everybody about it when visible.
96+
base.OnSizeChanged(performLayout);
97+
}
98+
}
99+
83100
/// <inheritdoc/>
84101
protected override void OnPalettePaint(object? sender, PaletteLayoutEventArgs e)
85102
{
@@ -181,25 +198,6 @@ public KryptonTaskDialogCommonButtonTypes CancelButton
181198
#endregion
182199

183200
#region Private
184-
private void OnSizeChanged(bool performLayout = false)
185-
{
186-
// Updates / changes are deferred if the element is not visible or until PerformLayout is called
187-
if (LayoutDirty && (Visible || performLayout))
188-
{
189-
// Button size is set through UpdateButtonSize() called from the constructor and OnPalettePaint.
190-
Panel.Height = _btnOk.Height + Defaults.PanelTop + Defaults.PanelBottom;
191-
192-
// Done
193-
LayoutDirty = false;
194-
195-
// Tell everybody about it when visible.
196-
if (!performLayout)
197-
{
198-
base.OnSizeChanged();
199-
}
200-
}
201-
}
202-
203201
private void SetupTableLayoutPanel()
204202
{
205203
_tlp = new()

‎Source/Krypton Components/Krypton.Toolkit/Controls Toolkit/KryptonTaskDialog/Elements/KryptonTaskDialogElementContent.cs‎

Lines changed: 27 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,31 @@ public KryptonTaskDialogElementContent(KryptonTaskDialogDefaults taskDialogDefau
5151
#endregion
5252

5353
#region Protected/Internal
54+
/// <inheritdoc/>
55+
protected override void OnSizeChanged(bool performLayout = false)
56+
{
57+
// Updates / changes are deferred if the element is not visible or until PerformLayout is called
58+
if (LayoutDirty && (Visible || performLayout))
59+
{
60+
Font font = _textControl.StateCommon.Font
61+
?? Palette.GetContentShortTextFont(PaletteContentStyle.LabelNormalControl, PaletteState.Normal)
62+
?? KryptonManager.CurrentGlobalPalette.BaseFont;
63+
64+
int height = TextRenderer.MeasureText(_textControl.Text, font, new SizeF(_textBoxWidth, float.MaxValue).ToSize(), textFormatFlags).Height;
65+
66+
// Controls seem to need a little help here to stay within the correct bounds
67+
Panel.Height = height + Defaults.PanelTop + Defaults.PanelBottom;
68+
_textControl.Width = _textBoxWidth - Defaults.PanelLeft - Defaults.PanelRight;
69+
_textControl.Height = height;
70+
71+
// Tell everybody about it when visible.
72+
base.OnSizeChanged(performLayout);
73+
74+
// Done
75+
LayoutDirty = false;
76+
}
77+
}
78+
5479
protected override void OnPalettePaint(object? sender, PaletteLayoutEventArgs e)
5580
{
5681
base.OnPalettePaint(sender, e);
@@ -64,12 +89,14 @@ protected override void OnPalettePaint(object? sender, PaletteLayoutEventArgs e)
6489
}
6590
}
6691

92+
/// <inheritdoc/>
6793
internal override void PerformLayout()
6894
{
6995
base.PerformLayout();
7096
OnSizeChanged(true);
7197
}
7298

99+
/// <inheritdoc/>
73100
public override bool Visible
74101
{
75102
get => base.Visible;
@@ -82,35 +109,6 @@ public override bool Visible
82109
}
83110
#endregion
84111

85-
#region Private
86-
private void OnSizeChanged(bool performLayout = false)
87-
{
88-
// Updates / changes are deferred if the element is not visible or until PerformLayout is called
89-
if (LayoutDirty && (Visible || performLayout))
90-
{
91-
Font font = _textControl.StateCommon.Font
92-
?? Palette.GetContentShortTextFont(PaletteContentStyle.LabelNormalControl, PaletteState.Normal)
93-
?? KryptonManager.CurrentGlobalPalette.BaseFont;
94-
95-
int height = TextRenderer.MeasureText(_textControl.Text, font, new SizeF(_textBoxWidth, float.MaxValue).ToSize(), textFormatFlags).Height;
96-
97-
// Controls seem to need a little help here to stay within the correct bounds
98-
Panel.Height = height + Defaults.PanelTop + Defaults.PanelBottom;
99-
_textControl.Width = _textBoxWidth - Defaults.PanelLeft - Defaults.PanelRight;
100-
_textControl.Height = height;
101-
102-
// Tell everybody about it when visible.
103-
if (!performLayout)
104-
{
105-
base.OnSizeChanged();
106-
}
107-
108-
// Done
109-
LayoutDirty = false;
110-
}
111-
}
112-
#endregion
113-
114112
#region Public
115113
/// <inheritdoc/>
116114
public string Text

‎Source/Krypton Components/Krypton.Toolkit/Controls Toolkit/KryptonTaskDialog/Elements/KryptonTaskDialogElementContentTest.cs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ internal override void PerformLayout()
9595
OnSizeChanged(true);
9696
}
9797

98-
private void OnSizeChanged(bool performLayout = false)
98+
protected override void OnSizeChanged(bool performLayout = false)
9999
{
100100
if (Visible || performLayout)
101101
{
@@ -120,7 +120,7 @@ private void OnSizeChanged(bool performLayout = false)
120120
Panel.Height += _description.GetDesiredVisibility() ? height + _description.Margin.Top + _description.Margin.Bottom : 0;
121121

122122
//// Tell everybody about it
123-
base.OnSizeChanged();
123+
base.OnSizeChanged(performLayout);
124124
}
125125
}
126126

‎Source/Krypton Components/Krypton.Toolkit/Controls Toolkit/KryptonTaskDialog/Elements/KryptonTaskDialogElementFooterBar.cs‎

Lines changed: 33 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,39 @@ public bool EnableExpanderControls
174174
#endregion
175175

176176
#region Overrides
177+
/// <inheritdoc/>
178+
protected override void OnSizeChanged(bool performLayout = false)
179+
{
180+
// Updates / changes are deferred if the element is not visible or until PerformLayout is called
181+
if (LayoutDirty && (Visible || performLayout))
182+
{
183+
// First set the size to zero so the images don't interfere with the tlp height
184+
_footNotePictureBox.Width = 0;
185+
_footNotePictureBox.Height = 0;
186+
_expanderButton.Width = 0;
187+
_expanderButton.Height = 0;
188+
189+
// update the images first to make them of the same size.
190+
UpdateFootNoteIcon();
191+
UpdateExpanderImage();
192+
193+
// and make'm visible
194+
_footNotePictureBox.Width = _tlp.Height;
195+
_footNotePictureBox.Height = _tlp.Height;
196+
_expanderButton.Width = _tlp.Height;
197+
_expanderButton.Height = _tlp.Height;
198+
199+
// Size the panel. Add an extra PanelBottom to gie it some more space at the border
200+
Panel.Height = _tlp.Height + Defaults.PanelTop + Defaults.PanelBottom + Defaults.PanelBottom;
201+
202+
// Tell everybody about it when visible.
203+
base.OnSizeChanged(performLayout);
204+
205+
// Done
206+
LayoutDirty = false;
207+
}
208+
}
209+
177210
/// <inheritdoc/>
178211
internal override void PerformLayout()
179212
{
@@ -226,41 +259,6 @@ public override Color BackColor2
226259
#endregion
227260

228261
#region Private
229-
private void OnSizeChanged(bool performLayout = false)
230-
{
231-
// Updates / changes are deferred if the element is not visible or until PerformLayout is called
232-
if (LayoutDirty && (Visible || performLayout))
233-
{
234-
// First set the size to zero so the images don't interfere with the tlp height
235-
_footNotePictureBox.Width = 0;
236-
_footNotePictureBox.Height = 0;
237-
_expanderButton.Width = 0;
238-
_expanderButton.Height = 0;
239-
240-
// update the images first to make them of the same size.
241-
UpdateFootNoteIcon();
242-
UpdateExpanderImage();
243-
244-
// and make'm visible
245-
_footNotePictureBox.Width = _tlp.Height;
246-
_footNotePictureBox.Height = _tlp.Height;
247-
_expanderButton.Width = _tlp.Height;
248-
_expanderButton.Height = _tlp.Height;
249-
250-
// Size the panel. Add an extra PanelBottom to gie it some more space at the border
251-
Panel.Height = _tlp.Height + Defaults.PanelTop + Defaults.PanelBottom + Defaults.PanelBottom;
252-
253-
// Tell everybody about it when visible.
254-
if (!performLayout)
255-
{
256-
base.OnSizeChanged();
257-
}
258-
259-
// Done
260-
LayoutDirty = false;
261-
}
262-
}
263-
264262
private void UpdateFootNoteIcon()
265263
{
266264
if (IconType != KryptonTaskDialogIconType.None)

‎Source/Krypton Components/Krypton.Toolkit/Controls Toolkit/KryptonTaskDialog/Elements/KryptonTaskDialogElementHeading.cs‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ public Color ForeColor
8585
public override Color BackColor1
8686
{
8787
get => base.BackColor1;
88+
8889
set
8990
{
9091
base.BackColor1 = value;
@@ -96,6 +97,7 @@ public override Color BackColor1
9697
public override Color BackColor2
9798
{
9899
get => base.BackColor2;
100+
99101
set
100102
{
101103
base.BackColor2 = value;

0 commit comments

Comments
 (0)