• Textboxes may be correctly disabled in designer.
  


• FirefoxNumericUpDown control has been added.
  


• NumericUpDown can be disabled.
  


• Textboxes forecolor is now updated when disabled.
  

• Pens and brushes are now disposed.
  


• TabControl text drawing is now aligned.
  


• TabControl is now supporting images.
  


• FirefoxH1 is now inheriting of Label.
  


• High quality drawing on FirefoxMainTabControl.
  


• Radiobuttons can't be unchecked.
  


• When a Radiobutton is checked, the other one in the parent are automaticaly unchecked.
  

1. Your Helpers.RoundRect() method is long and messy, it could be greatly improved by using a GraphicsPath
   


2. Declarations inside the paint event are a bad habit. Basically, every time you paint your control, you're redeclaring those objects. Place your declarations in the class scope and assign their values where you need them.
   


3. Dispose of any objects that implement IDisposable, such as fonts, brushes, and pens. Disposing of these objects will save you some RAM.
   


4. I noticed you had two separate controls for your headers. One inherits from Control while the other inherits from Label. Inheriting from Label is probably your best option here. I don't think you really need two separate controls.
   


5. I see you've made an extra property called Enable in your controls. A better option would be to overload the Enabled property. This way you don't have two very similar properties.
   


6. In your tab control, you're setting the backcolor of your tabs in the OnPaint() event. This is processor intensive. Use the OnControlAdded() method instead.
   

(Jul 25, 2015 12:30 PM)Hawk HF Wrote:  Excellent. Your work is always improving.

The visual quality of your themes are excellent, but I think you should take some time to work on your code.

From what I can see, your Helpers module consists mostly of snippets from other users here, including me. I see it in your controls as well. I appreciate that you're giving credits, but I honestly believe you're at the point in GDI where you should be able to code your own methods and functions.

I noticed a few things that you might want to fix:

1. Your Helpers.RoundRect() method is long and messy, it could be greatly improved by using a GraphicsPath
   


2. Declarations inside the paint event are a bad habit. Basically, every time you paint your control, you're redeclaring those objects. Place your declarations in the class scope and assign their values where you need them.
   


3. Dispose of any objects that implement IDisposable, such as fonts, brushes, and pens. Disposing of these objects will save you some RAM.
   


4. I noticed you had two separate controls for your headers. One inherits from Control while the other inherits from Label. Inheriting from Label is probably your best option here. I don't think you really need two separate controls.
   


5. I see you've made an extra property called Enable in your controls. A better option would be to overload the Enabled property. This way you don't have two very similar properties.
   


6. In your tab control, you're setting the backcolor of your tabs in the OnPaint() event. This is processor intensive. Use the OnControlAdded() method instead.
   

These are just a few things I found while skimming through your source.
Hopefully you take some time to improve upon some of these areas.

Good work and keep it up!