View Issue Details

IDProjectCategoryView StatusLast Update
0005439JEDI VCL00 JVCL Componentspublic2011-09-21 14:10
ReportertetarddAssigned ToAHUser 
PrioritynormalSeveritymajorReproducibilityalways
Status resolvedResolutionfixed 
Product Version3.40 
Target VersionFixed in Version3.45 
Summary0005439: 2 serious flaws in TJvStringGrid.AutoSizeCol
Description- TJvStringGrid.AutoSizeCol add 8 to the text width. That should not be hard-coded, what if someone wants another padding value?
- If the canvas has been changed to draw, for example fixed rows (SetCanvasProperties), the calculated text width is wrong. See the example.

A minor issue (just because I was told off for not following the formatting of your code, I just thought I'd report one): unconsistent declaration:
  TJvStringGrid = class(TJvExStringGrid)
  private
    ...
    FSetCanvasProperties: TDrawCellEvent;
    FGetCellAlignment: TGetCellAlignmentEvent;

Should that not be:
  TJvStringGrid = class(TJvExStringGrid)
  private
    FOnSetCanvasProperties: TDrawCellEvent;
    FOnGetCellAlignment: TGetCellAlignmentEvent;
TagsNo tags attached.

Activities

2010-12-11 12:43

 

Bug jvStringGrid.zip (615,878 bytes)

AHUser

2011-02-16 20:40

developer   ~0018385

Fixed in svn revision 12987.

Issue History

Date Modified Username Field Change
2010-12-11 12:43 tetardd New Issue
2010-12-11 12:43 tetardd File Added: Bug jvStringGrid.zip
2011-02-15 13:46 obones Status new => acknowledged
2011-02-16 20:40 AHUser Note Added: 0018385
2011-02-16 20:40 AHUser Status acknowledged => resolved
2011-02-16 20:40 AHUser Fixed in Version => Daily / SVN
2011-02-16 20:40 AHUser Resolution open => fixed
2011-02-16 20:40 AHUser Assigned To => AHUser
2011-09-21 14:10 obones Fixed in Version Daily / SVN => 3.45