Project

General

Profile

Actions

Feature #1139

closed

WPieChart lacks a feature to avoid rendering of labels if the percentage is too small

Added by Stefan Ruppert over 12 years ago. Updated about 12 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Pieter Libin
Target version:
Start date:
01/18/2012
Due date:
% Done:

80%

Estimated time:

Description

Within a pie chart label texts are rendered even if there is no space left to render a label text. Therefore the label text is overwritten by another label text. This only occurs on segments which are really small.

Therefore I added a new member called avoidLabelRendering_ which stores a percentage value which will avoid label rendering if a segment is below that percentage. See attachments.

I would like to see this patch really soon integrated into Wt, since we really need this feature in our upcoming web application.

Regards,

Stefan


Files

WPieChartAvoidRendering.patch (2.15 KB) WPieChartAvoidRendering.patch The patch file implementing the avoid label rendering Stefan Ruppert, 01/18/2012 08:30 PM
Wt-PieChart-AvoidRendering-Before.png (13.6 KB) Wt-PieChart-AvoidRendering-Before.png A sample pie chart without the patch Stefan Ruppert, 01/18/2012 08:30 PM
Wt-PieChart-AvoidRendering.png (12.7 KB) Wt-PieChart-AvoidRendering.png A sample pie chart with the patch Stefan Ruppert, 01/18/2012 08:30 PM
Actions #1

Updated by Koen Deforche over 12 years ago

  • Status changed from New to InProgress
  • Assignee set to Pieter Libin
  • Target version set to 3.2.1

Hey Stefan, Pieter,

I think this is indeed a simple and robust solution.

Pieter can you review the patch and apply ?

Regards,

koen

Actions #2

Updated by Pieter Libin over 12 years ago

  • Status changed from InProgress to Resolved

I applied your patch to our git repository,

I changed the documentation about the default value (said it was 5.0 while it was 0.0 in the code).

Actions #3

Updated by Koen Deforche about 12 years ago

  • Status changed from Resolved to Closed

Fixed in 3.2.1

Actions

Also available in: Atom PDF