[HeDiBo]

'Support' wrote:

Hello!

Do you mean that internal skins are not represented in the SkinName popup list?

No the addition (internal) is lost

[HeDiBo]

Also the font settings of sSkinProvider.AddedTitle are not honored.

I think with the changes in scaling these font problem have crept in.

[HeDiBo]

It's still totally wrong in AC 14.12

Please change the code to

Code:

function TcxACLookAndFeelPainter.DefaultContentEvenColor: TColor;
begin
Result := DefaultManager.Palette[pcEditBG_EvenRow]
end;

function TcxACLookAndFeelPainter.DefaultContentOddColor: TColor;
begin
Result := DefaultManager.Palette[pcEditBG_OddRow]
end;

[HeDiBo]

It's totally unusable now (AC 14.11)

None of the grids show alternating colors, no matter whether style is specified or not!!

Just return it to what it was. I found another way by using a TsListView in that particular case.

You can simplify this:

Code:

function TcxACLookAndFeelPainter.DefaultContentEvenColor: TColor;
begin
if Skinned then
Result := DefaultManager.Palette[pcEditBG_EvenRow]
else
Result := inherited DefaultContentEvenColor;
end;

function TcxACLookAndFeelPainter.DefaultContentOddColor: TColor;
begin
if Skinned then
Result := DefaultManager.Palette[pcEditBG_OddRow]
else
Result := inherited DefaultContentOddColor;
end;

To this:

Code:

function TcxACLookAndFeelPainter.DefaultContentEvenColor: TColor;
begin
Result := DefaultManager.Palette[pcEditBG_EvenRow]
end;

function TcxACLookAndFeelPainter.DefaultContentOddColor: TColor;
begin
Result := DefaultManager.Palette[pcEditBG_OddRow]
end;

because the functions are not called if skinned false

[HeDiBo]

'Support' wrote:

Thank you for the message. The issue will be solved in the nearest release.

The leak is at the end of TsSkinManager.Destroy:

Code:

SetLength(ThirdLists, 0);
FreeAndNil(FThirdParty);
FreeAndNil(sc); (*** DB ***)
inherited Destroy;

The marked statement should be added

[HeDiBo]

'Support' wrote:

Yes, I see, this way doesn't work.

Result of “inherited DefaultContentEvenColor” not clWindow and not clDefault, so condition is False always.

I have another idea, try the attached file. But this solution works only if clDefault specified in Style.

With clWindow it doesn't work.

Thanks for your effort, but that changes coloring in all grids, not just in the grid where the style was defined.

[HeDiBo]

'Support' wrote:

I have looked it in the VCL and haven't idea how it may be implemented.

Disabled control doesn't catch any mouse events and WM_NCHITTEST message even.

It's a well known problem.

The solution suggested on the internet is to use the message pump of the containing form to simulate the hint addressed to the disabled control.

[HeDiBo]

In AC 14.07 you test the color on clDefault OR clWindow.

If ContentOdd and ContentEven are both set to the style with the clWindow color, this is the result:

[attachment=9010:cxGridColor.jpg]

The left grid has made this change, the right hand one stays away from the styles logic.

If I change the style color to clDefault, all's well again, but the rows show alternating colors again.

[HeDiBo]

Problem solved in 14.07 🌺

[HeDiBo]

'Support' wrote:

This behavior will be improved in the v14.07 at the nearest days.

All's well now 🆗

[HeDiBo]

Do you hesitate to change the version property to a string?

It's still a float in 14.07

[HeDiBo]

Could you have a look at it, please ❓

[HeDiBo]

'Support' wrote:

I'm sorry for the error, please try the attached file.

Whole package will be updated at the nearest days.

Thanks.

I'm not seeing a switch to using the internal skin version as a string. That would make it compatible with the skinmanager's skin version.

There's no chance developers will get into trouble because the version of internal skins never worked before, so no one will have been using it.

[HeDiBo]

'Support' wrote:

I will try to make it in the v14.05

In 14.06 the ghosting is fixed

However the arrows still point the wrong way in these cases:

• Align = alTop, Placement = asBottom

[*] Align = alLeft, Placement = asRight[*]Align = alBottom, Placement = asTop[*]Align = alRight, Placement = asLeft

In other words, if the Placement is the opposite of Align, the arrow should be reversed too.

[HeDiBo]

'Support' wrote:

I will try to do it in the v14.06

It's not fixed at all in 14.06

To ensure you understand the severity of this: in this version of AC I cannot add any internal skin or refresh them.

I found there are three places in sInternalSkins.pas where this problem occurs.

Again: switch to a string representation of the version. That makes it compatible with the the other occurrence of the version property, allowing constructs like '13.05A'.

[HeDiBo]

'Fess' wrote:

Hello! Problem is NOT solved in 14.06 ((

It's only partly solved. Now I can select a skin with a changed name from the drop down list.

But adding a skin to the Internal skins still fails on a floating point error.

So does refreshing them

[HeDiBo]

'Support' wrote:

We haven't access to the control or his properties.

DevExpress skin engine calls a function where requests a color and we can return a color only:

Code:

function TcxACLookAndFeelPainter.DefaultContentEvenColor: TColor;
begin
if Skinned then
Result := DefaultManager.Palette[pcEditBG_EvenRow]
else
Result := inherited DefaultContentEvenColor;
end;

You might do something like this:

Code:

function TcxACLookAndFeelPainter.DefaultContentEvenColor: TColor;
begin
Result := inherited DefaultContentEvenColor;
if Skinned then begin
if Result =
then Result := DefaultManager.Palette[pcEditBG_OddRow]
else Result := DefaultManager.Palette[pcEditBG_EvenRow];
end;
end;

[HeDiBo]

'Support' wrote:

I haven't idea how to change it, unfortunately.

AlphaControls just gives a color there when color is requested. And can't affect to logic.

It could be done by specifying a very special color in the Styles.ContentEven and Styles.ContentOdd properties of the TcxGridDBTableView.

AC would have to interpret that special value as the EDIT color or any similar color that would work also for dark skins.

[HeDiBo]

'Support' wrote:

I can't repeat it. On your side this issue is repeated always?

Here's a project that does not allow selection of internal skins at design time:

[attachment=8976:acBitBtnHints.zip]

I think the problem is the old format internal skins, that I cannot renew because of the floating point conversion error that I receive.

[HeDiBo]

'Support' wrote:

Do you mean a changing of SkinName in design-time?

yes

[Author]

Posts