Lazarus

Miscellaneous => Suggestions => LCL => Topic started by: Zaher on May 24, 2009, 03:14:59 pm

Title: BidiMode improvments patch
Post by: Zaher on May 24, 2009, 03:14:59 pm

I know i touch a sensitive code but we can discuss about it.

Added TVisualCompont it is between TLCLComponent and and TControl, it has a properties and methods that have visual effect but still it unvisible at form. i will use it for BidiMode and ParentBidimode
TMenu, TPopupMenu, TScreen, TTrayIcon and TPopupNotifier and any component need bidimode , TTheme and TToolbar...

Now it can detect if the Component was inherited from TVisualComponent to make loop when BidiMode changed in the form/control.

Remove duplicating of implement the same code in both TControl and TMenu and every TComponent need the BidiMode.

I kept UseRightToLeftScrollBar in TControl

Added BidiMode to TApplication.

TApplicationProperties must *not* inherite it from TVisualComponent it is like TApplication have no ParentBidiMode

I used the forum for my patch, because i like to edit my posts, but i can use Mail-List if you wish.

Title: Re: BidiMode improvments patch
Post by: Paul Ishenin on May 24, 2009, 04:47:29 pm

I would leave class hierarchi as is. No idea what other developers think.

Title: Re: BidiMode improvments patch
Post by: Zaher on May 24, 2009, 04:58:44 pm

I can merge of BidiMode the into TLCLComponent.

Title: Re: BidiMode improvments patch
Post by: Vincent Snijders on May 24, 2009, 07:11:48 pm

For discussion with all developers, the mailing list is a better platform.

Title: Re: BidiMode improvments patch
Post by: Zaher on May 24, 2009, 11:20:48 pm

Ok, I will repost the patch and my post in mail the list.

Title: Re: BidiMode improvments patch
Post by: Zaher on May 25, 2009, 05:11:11 pm

I posted to mail list and there is a changes in the patch, and more touch sensitive code :P .

Thanks