LoadDocument updates screen cursor

[GoogleCodeExporter]

Before actually loading a document LoadDocument stores the current values of 
Screen.Cursor and sets Screen.Cursor to crHourGlass, it then restores the old 
value once the document has loaded.

I do not believe this is desired behaviour (but, since this may be a matter of 
opinion, please excuse me if I waffle on a bit with some justification)

One of my projects performs 1/2 second refreshes of a page loaded from memory; 
this is extremely fast, but still causes the screen cursor to momentarily 
change to crHourGlass.

Whilst the current behaviour may be desirable in many situations, there is no 
way to turn it off.

If the screen cursor update were to be removed from LoadDocument people wanting 
that behaviour could implement it themselves, when specifically required, in a 
similar fashion to the current implementation. 

In general, I don't think a VCL component should be changing any setting 
outside its own scope.

Just my $0.02, thanks for listening...

Original issue reported on code.google.com by wildcrof...@gmail.com on 15 Jun 2014 at 10:39

[GoogleCodeExporter]

Thanks for this suggestion.

r479 adds a published property named LoadCursor to THtmlViewer, TFrameViewer 
and TFrameBrowser to control the Screen.Cursor THtmlVIewer.LoadDocument. 

Its default is crHourGlass. Setting LoadCursor to crDefault does not touch 
Screen.Cursor while loading. Other values set the selected cursor.

Original comment by OrphanCat on 22 Jul 2014 at 10:00

• Changed state: Fixed

[GoogleCodeExporter]

Original comment by OrphanCat on 22 Jul 2014 at 10:01