Results 1 to 5 of 5

Thread: AggregationRowConfig.setCellStyle is beeing used for all columns

Hybrid View

Previous Post Previous Post   Next Post Next Post
    Success! Looks like we've fixed this one. According to our records the fix was applied for EXTGWT-2755 in 3.0.4.
  1. #1
    Sencha User
    Join Date
    Mar 2012
    Posts
    88
    Vote Rating
    3
      0  

    Default AggregationRowConfig.setCellStyle is beeing used for all columns

    Version(s) of Ext GWT
    Ext GWT 3.0.1

    Browser versions and OS (and desktop environment, if applicable)
    IE 9 Windows 7

    Virtual Machine
    No

    Description
    I'm using an aggregation grid. I want some of the summaries to have text-align = left, and some text-align = right. Using AggregationRowConfig.setCellStyle(ColumnConfig, style) it should be possible to define different styles for different columns. Instead, all of the styles are beeing applied to all columns.

    In my test case both columns are aligned right. You can even coment out either the first or the second style line - both columns get the defined style.

    Run mode

    Development Mode

    Test case

    Code:
    AggregationRowConfig<T> agg = new AggregationRowConfig<T>();
    agg.setRenderer(column1, new SummarySafeHtmlRenderer<T>("column1 summary"));
    agg.setRenderer(column2, new SummarySafeHtmlRenderer<T>("column2 summary"));
      
    agg.setCellStyle(column1, MY_TEXTALIGN_RIGHT);
    agg.setCellStyle(column2, MY_TEXTALIGN_LEFT);

  2. #2
    Sencha User
    Join Date
    Mar 2012
    Posts
    88
    Vote Rating
    3
      0  

    Default

    Any updates?

  3. #3
    Ext GWT Premium Member
    Join Date
    Apr 2008
    Location
    Canada
    Posts
    14
    Vote Rating
    1
      0  

    Default

    I can confirm this bug still exists in GXT 3.0.3.

    The problem is in the doRender() method of the ColumnFooter class. The cellClass variable is not being reset inside the "loop each cell per row" for loop. Should be a simple fix.

  4. #4
    Sencha User
    Join Date
    Feb 2009
    Location
    Minnesota
    Posts
    2,737
    Vote Rating
    93
      0  

    Default

    Thanks, you're absolutely correct - we fixed a similar issue a while ago, but apparently missed this one.

  5. #5
    Sencha User
    Join Date
    Feb 2009
    Location
    Minnesota
    Posts
    2,737
    Vote Rating
    93
      0  

    Default

    This has been fixed in SVN and the latest nightly build, and will be available in the next release.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •