BidiMode improvments patch
lazarus / 7-LCL

----------------------------------------------------------------------
Zaher · 2009-05-24 13:14

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.

----------------------------------------------------------------------
Paul Ishenin · 2009-05-24 14:47

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

----------------------------------------------------------------------
Zaher · 2009-05-24 14:58

I can merge of BidiMode the into TLCLComponent.

----------------------------------------------------------------------
Vincent Snijders · 2009-05-24 17:11

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

----------------------------------------------------------------------
Zaher · 2009-05-24 21:20

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

----------------------------------------------------------------------
Zaher · 2009-05-25 15:11

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

Thanks