Delphi Berlin 10.1 ScaleBy TPageControl

[Sourcemaker]

Hallo,

habe gerade 10.1 Berlin installiert und stelle fest das ScaleBy die Controls
die auf einem PageControl sind erneut bzw. mehrfach skaliert.

Hat das auch schon jemand festgestellt?

Grüße

Frank

[gullatrulla]

Kann ich bestätigen... Problem auch bei mir.
Ein neues Problem von "Vielen".

Bleibe vorläufig auf XE8.

Gruss
Roland

[Rollo62]

Zitat:

Rufen Sie ScaleControls auf, um nur die untergeordneten Komponenten des Steuerelements zu skalieren, während die Größe dieses Steuerelements unverändert bleibt. Die Bedeutung der Parameter entspricht denen der Methode ScaleBy.

Könnte es sein das man beides machen muss um zum Ergebnis zu kommen ?
Wäre ja ein gutes Feature.

Rollo

[jaenicke]

Der Fehler liegt hier:

Delphi-Quellcode:

procedure TPageControl.ChangeScale(M, D: Integer; isDpiChange: Boolean);
var
  I: Integer;
begin
  inherited ChangeScale(M, D, isDpiChange);
  for I := 0 to FPages.Count - 1 do
    TTabSheet(Fpages[I]).ChangeScale(M, D, isDpiChange);
end;

In dem inherited Aufruf werden die TabSheets bereits skaliert, da diese in Controls des PageControls drin sind und somit von ScaleControls erwischt werden. Die zusätzliche Skalierung muss raus...

Leider lässt sich das auch nicht ganz so einfach fixen... man muss dafür die VMT verbiegen, quasi das Überschreiben der Methode in TPageControl rückgängig machen. Dann sollte alles funktionieren.

// EDIT:
Es gibt bereits eine entsprechende Bugmeldung, die ich soeben mit den Details ergänzt habe:
https://quality.embarcadero.com/browse/RSP-14750

[mschumann]

Danke für den Hinweis auf die Fehlerstelle. Man kann ja vorübergehend eine Kopie der vcl.comctrls.pas korrigieren und in den eigenen Quellcodeordner tun, dann klappt das. Bei mir klappt das.

Ich habe noch eine seltsame Sache bei der Toolbar gefunden, die beim Skalieren falsche Buttons anzeigt, das geht weg, wenn man das Attribut FLAT ausschaltet.

[Sourcemaker]

Habe ich ebenfalls so gelöst.

Grüße

Frank

[Olli4362]

Hi,

könnt ihr etwas genauer erklären, wie ihr das Problem gelöst habt?
Die vcl.ComCtrls.pas habe ich in mein Projektverzeichnis kopiert und Zeilen herausgenommen

Delphi-Quellcode:

procedure TPageControl.ChangeScale(M, D: Integer; isDpiChange: Boolean);
var
  I: Integer;
begin
  inherited ChangeScale(M, D, isDpiChange);
//  for I := 0 to FPages.Count - 1 do
//    TTabSheet(Fpages[I]).ChangeScale(M, D, isDpiChange);
end;

DCU wird erstellt, aber es ändert sich an dem Fehler nichts.

Olli

[sh17]

Nur zur Zwischeninfo, die Zeilen sehen in Delphi Berlin Upd 1 immer noch so aus. Wurde also anscheinend nichts behoben.

[Sourcemaker]

Hi Olli,

Zitat:

könnt ihr etwas genauer erklären, wie ihr das Problem gelöst habt?
Die vcl.ComCtrls.pas habe ich in mein Projektverzeichnis kopiert und Zeilen herausgenommen

soweit ich das übersehen kann ist das alles was gemacht werden muss.

Eventuell nochmal ein Breakpoint gesetzt ob das Programm tatsächlich die Unit nutzt?

Du müsstes beim Projekt erzeugen ja auch einen Hinweis bekommen das die Variable i ungenutzt ist.:-D

Gruß

Frank

[Olli4362]

Danke für die Info!

Ich habe das noch mal genauer untersucht:
Es funktioniert (zumindest nach dem ersten Update von Delphi 10.1) richtig.

Leider nicht immer:
Ist auf dem Tabsheet eine Element, dass beim Alignment alRight gesetzt hat, dann sind die Elemente zu breit.

Vermutlich wird die Breite schon durch das Aligment auf den richtigen Wert geändert, und dann die Breite mit ChangeScale nochmals angepasst. Dann ragt es drüber.

Dabei hilft auch leider das komplette Entfernen der Zeilen in TPageControl.ChangeScale nicht.

in der VCL.Controls gibt es noch

Delphi-Quellcode:

procedure TWinControl.ChangeScale(M, D: Integer; isDpiChange: Boolean);
begin
  [B]DisableAlign[/B];
  try
    if not isDpiChange then
      ScaleControls(M, D);
    if (M <> D) and (csReading in ComponentState) and
      (sfDesignSize in ScalingFlags) then
    begin
      FDesignSize.X := MulDiv(FDesignSize.X, M, D);
      FDesignSize.Y := MulDiv(FDesignSize.Y, M, D);
    end;
    inherited ChangeScale(M, D, isDpiChange);
    ScalePadding(M, D);
  finally
    [B]EnableAlign[/B];
  end;
end;

Scheint aber für die Tabsheets nicht zu funktionieren.

[Olli4362]

So, habe mal ein Beispielprojekte erstellt.

Das demonstriert sehr schön was passiert. Darin das Original in 96 DPI und dann die Probleme mit dem Tabsheet bei 120 DPI und das Projekt zum Nachvollziehen.

Interessanterweise löst sich das Problem, wenn man alle Elemente im Tabsheet vorher auf ein TPanel legt!

Falls einer weiß, wie man eine Bugreport schickt, bitte ich um Hinweise.

Kennt ihr eine Delphi-Version, die das noch richtig macht? Man kann ja auch die alten Versionen herunterladen...

[Olli4362]

Habe es gefunden:

VCL.ComCtrls.pas in das eigene Projektverzeichnis kopieren


procedure TTabSheet.UpdateControlOriginalParentSize(AControl : TControl;
var AOriginalParentSize: TPoint);
begin
inherited;
exit; // hier muss ein EXIT rein, dann geht es!
if (csLoading in ComponentState) then
begin
AOriginalParentSize.X := FDesignSize.X;
AOriginalParentSize.Y := FDesignSize.Y;
end;
if not (csReading in ComponentState) and not HandleAllocated then
Dec(AOriginalParentSize.X, BorderWidth * 2);
end;

Compile and smile!

[jaenicke]

Dann kannst du das ja hier als Bug melden, damit es hoffentlich bald auch ohne Workaround geht:
https://quality.embarcadero.com

[Olli4362]

Bugreport ist geöffnet. Wäre nett, wenn ihr voted!

https://quality.embarcadero.com/browse/RSP-15974