Doppelte Scrollbars bei Lazreport-Vorschau, Bug?

[Soner]

Ich aktualisiere gerade mein Lazarus von Version 2.2. zu 3.4.
Jetzt hat LazReport-Vorschau doppelte Scrollbars, ich bin der Einziger, hat niemand sonst diesen Fehler?

Vorschau

lazreportdoppelterscrollbars.jpg (43.44 KiB) 1595 mal betrachtet

Bildschirmfoto ist vom Beispielprogramm \Lazarus\components\lazreport\samples\stringgrid.

Lazarus 3.4, fpc 3.2.2 beide 32 Bit. Windows 10.

[kralle]

Moin,

Ich würde darauf tippen, daß der äußere Scrollbalken von der Anwendung stammt, weil die auf einer höheren Auflösung erstellt wurden und der innere von dem Dokument, das für die Anzeige zu groß ist.

Gruß Heiko

[Soner]

Beides gehört zu TfrPreview, innere Scrollbars gehörren zu Scrollbox, äußere zu zusätzlichen Scrollbars.

Ich habe jetzt Lazreport von alten Lazarus in den neuen kopiert, dort tauchte der gleiche Fehler, also Lazreport scheint OK zu sein. Der Fehler muss von LCL sein.

Ich hatte vergessen zu erwähnen, dass ich gestern Lazarus Fixes_3_0 Branch heruntergeladen habe und es verwende. Ich kontrolliere es nochmal mit 3.4.

[Soner]

Lazarus 3.4 und Lazarus Entwickler-Version hat das Problem auch. Irgendwo muss bei LCL geändert worden sein, dass die Scrollbars vom inneren Scrollbox sichtbar wird.
Schade, ich muss jetzt weiter 2.2. verwenden.

Bin ich der Einziger, der diese Komponente verwendet?

[Soner]

Ich habe die Lösung gefunden, bisher scheint keine Seiteneffekte zu geben.
Die Lösung ist die Klasse von ScrollBox1 von TScrollBox zu TPanel zuändern.
Viel gesucht und geschrieben einfache Lösung.

[kralle]

Moin,

Hast Du einen Bug-Report geschrieben.

Gruß Heiko

[Soner]

Ich glaube für den Bugreport sollten einige Leute den Fehler bestätigen, z.B. das Beispiel aus dem ersten Beitrag benutzen.
Meine Vermutung ist, dass in LCL, besonders bei TScrollbox etwas verändert wurde.
Vielleicht funktionierte früher TScrollbox nicht wie Delphi, deshalb wurden die beiden zusätzlichen Schrollbars eingeführt. Früher waren die Scrollbars von TScrollbox in Previewform nicht sichtbar.
Bei LR_View.pas wurde kaum etwas geändert.

EDIT:
Vielleicht ist das gar kein Fehler sondern frühere falsche Benutzung wurde jetzt sichtbar.

[Soner]

Es wäre gut, wenn einige Leute das Beispiel aus dem ersten Beitrag benutzt und berichtet ob der Fehler bei ihm auch auftaucht.
Wenn der Fehler nur bei mir auftaucht, dann ist ein Bugreport oder Patch sinnlos.

[charlytango]

Nachdem bei mir noch einige Lazarus-Versionen auf der Platte liegen habe ich mal einen Test gemacht.
Als Testprojekt wurde das in der jeweiligen Lazarus-Version befindliche Projekt in \components\lazreport\samples\stringgrid verwendet.

Der beschriebene Fehler taucht auf in

3.4
3.2
nicht in 2.2.6 (weiter runter habe ich nicht getestet)

Laz34.png (153.84 KiB) 1480 mal betrachtet

Laz32.png (112.69 KiB) 1480 mal betrachtet

Laz226.png (121.21 KiB) 1480 mal betrachtet

Als Gegenprobe habe ich noch das Sample Project aus der 3.4 Version mit Laz 2.2.6 geöffnet und nach einem Clean Build getestet.
-> kein Fehlverhalten.
Es liegt wohl eher an der Lazarus-Verion als am Testcode.

[djdhg]

So auch in:
Lazarus 3.99

LazRep.png (166.46 KiB) 1448 mal betrachtet

Lg

[Soner]

Vielen dank, also unter Windows gibt es auf jeden Fall diesen Fehler.

Ich lade jetzt Beispielprogramm hoch mit den korrigierten "lr_view.pas" und "lr_view.lfm" Dateien.
Könnt ihr bitte diese beiden Dateien in den Ordner C:\lazarus\components\lazreport\source kopieren und das Beispielprogramm laufen lassen und berichten.
Können vielleicht auch Linux- und Mac-Leute testen?
Vorher eure Dateien umbennen und damit ihr es wieder rückgängig machen könnt. Keine Angst euer Lazarus geht nicht kaputt.
Reportvorschau müsste sich wie gewohnt verhalten und aussehen.

Vielen Dank.

[djdhg]

Sieht gut aus

Laz2.png (178.38 KiB) 1416 mal betrachtet

[paweld]

@Soner: Vielen Dank für die Korrektur, Ihre Änderungen lösen das Problem. Nur die Bearbeitung in Lazarus verdeckt die notwendigen Änderungen, weil sie die Reihenfolge einiger Eigenschaften ändert. Es ist ein Patch beigefügt, der nur die notwendigen Änderungen enthält.

Bitte melden Sie den Fehler im Bugtracker mit einem Fix.

[Soner]

Danke paweld. Ich habe jetzt Bugreport mit Patch eingereicht.
Ich habe auch beim rechten und unteren Panel, enthalten Scrollbars, autosize:=true gemacht.