Absolutely Awfull grid titles in DevEx cxGrid

Viewing 9 posts - 1 through 9 (of 9 total)
  • Author
    Posts
  • #42903
    HeDiBo
    Participant

      I changed the code in acLFPainter in TcxACLookAndFeelPainter.DrawHeader slightly:

      CODE
      Section := {$IFDEF ALTHEADER} s_MainMenu {$ELSE} s_ColHeader {$ENDIF};
      AOnDrawBackground := nil;

      TmpBmp := CreateBmp24(WidthOf(ABounds), HeightOf(ABounds));
      i := PaintSection(TmpBmp, Section, s_Button, State, DefaultManager, ABounds.TopLeft, DefaultManager.GetGlobalColor);
      BitBlt(ACanvas.Handle, ABounds.Left, ABounds.Top, TmpBmp.Width, TmpBmp.Height, TmpBmp.Canvas.Handle, 0, 0, SRCCOPY);
      FreeAndnil(TmpBmp);

      Note that the original code did not use the value from the Section variable in the PaintSection call, but used the string s_ColHeader. Please correct that in the next release, otherwise setting the Section variable has no function.

      The alternative header skin s_MainMenu works quite well with almost all of the skins. But you may have a better section that I can use, to make it even better compatible with the standard grid headers.

      Please, change the code in this way, just to keep me happy

      #42917
      Support
      Keymaster

        This problem exists because the Sand skin have not a section for headers drawing (COLHEADER).
        I think, maybe will be better just define this section in skin?

        PS. Could you show a screenshot, what you have now?

        #42924
        HeDiBo
        Participant

          QUOTE (Support @ Jul 16 2010, 11:02 AM) <{POST_SNAPBACK}>
          This problem exists because the Sand skin have not a section for headers drawing (COLHEADER).
          I think, maybe will be better just define this section in skin?

          PS. Could you show a screenshot, what you have now?


          Adding that section would certainly do the trick. Could you implement that?

          The screenshot was already in the first part of this topic, together with a test project.

          #42925
          Support
          Keymaster

            I have updated this skin. Try it, please.

            #42927
            HeDiBo
            Participant

              QUOTE (Support @ Jul 16 2010, 03:16 PM) <{POST_SNAPBACK}>


              Works Wonderful

              #42928
              Support
              Keymaster

                Skin for version 6 is attached

                #42931
                HeDiBo
                Participant

                  QUOTE (Support @ Jul 16 2010, 03:38 PM) <{POST_SNAPBACK}>
                  Skin for version 6 is attached Most of the times it looks great! There are still some minor issues. I'll come back to that next week. Thank you for your quick solution!
                  #42954
                  HeDiBo
                  Participant

                    QUOTE (HeDiBo @ Jul 16 2010, 03:49 PM) <{POST_SNAPBACK}>
                    There are still some minor issues.


                    I have included a new picture, to show you the problems in AC 6.68:

                    [attachment=4050:Minor_he…problems.jpg]

                    Clockwise I find the following issues:

                    1. The headers of a standard TDBGrid look like clickable buttons. But they are not clickable.
                    2. The box around the TsDBGrid is missing in first instance. Once focused, the box appears again.
                    3. The triangular indicator in a TsDBGrid is way too small.
                    4. The column headers of a TcxGrid are clickable, but appear as if they are not.
                    5. There is a separator-line missing between the headers and the “GroupByBox” of a TcxGrid.

                    So, really the only grid that's completely correct is the TwwDBGrid dry.gif

                    #42961
                    Support
                    Keymaster

                      I'll try to improve it soon

                      Author
                      Posts
                    Viewing 9 posts - 1 through 9 (of 9 total)
                    • You must be logged in to reply to this topic.