HG_REPOSITORIES/LPP/INSTRUMENTATION/USERS/JEANDET/qtcharts Commit - r737:e81beeb51921

Code review: Fixed simple issues in Bar and Legend

Tero Ahola -

r737:e81beeb51921

parent child

Context file:

r737:e81beeb51921

Collapse all files

src/barchart/bar.cpp +5 -4

             #include "bar_p.h"
             #include <QDebug>
             #include <QPainter>
             #include <QGraphicsSceneEvent>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             Bar::Bar(QString category, QGraphicsItem *parent)
                 : QGraphicsRectItem(parent),
                 mCategory(category)
             {
-                setAcceptedMouseButtons(Qt::LeftButton|Qt::RightButton);
+                setAcceptedMouseButtons(Qt::LeftButton | Qt::RightButton);
                 setAcceptHoverEvents(true);
             }
-            void Bar::mousePressEvent(QGraphicsSceneMouseEvent* event)
+            void Bar::mousePressEvent(QGraphicsSceneMouseEvent *event)
             {
                 if (event->button() == Qt::LeftButton) {
                     emit clicked(mCategory);
                 } else if (event->button() == Qt::RightButton) {
                     emit rightClicked(mCategory);
                 }
             }
-            void Bar::hoverEnterEvent(QGraphicsSceneHoverEvent* event)
+            void Bar::hoverEnterEvent(QGraphicsSceneHoverEvent *event)
             {
                 emit hoverEntered(event->lastScreenPos());
             }
-            void Bar::hoverLeaveEvent(QGraphicsSceneHoverEvent* /*event*/)
+            void Bar::hoverLeaveEvent(QGraphicsSceneHoverEvent *event)
             {
+                Q_UNUSED(event)
                 emit hoverLeaved();
             }
             #include "moc_bar_p.cpp"
             QTCOMMERCIALCHART_END_NAMESPACE

src/barchart/bar_p.h +1 -1

             #ifndef BAR_H
             #define BAR_H
             #include "qchartglobal.h"
             #include <QGraphicsRectItem>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             // Single visual bar item of chart
             class Bar : public QObject, public QGraphicsRectItem
             {
                 Q_OBJECT
             public:
-                Bar(QString category, QGraphicsItem *parent=0);
+                Bar(QString category, QGraphicsItem *parent = 0);
             public:
                 void mousePressEvent(QGraphicsSceneMouseEvent *event);
                 void hoverEnterEvent(QGraphicsSceneHoverEvent *event);
                 void hoverLeaveEvent(QGraphicsSceneHoverEvent *event);
             Q_SIGNALS:
                 void clicked(QString category);
                 void rightClicked(QString category);
                 void hoverEntered(QPoint pos);
                 void hoverLeaved();
             private:
                 QString mCategory;
             };
             QTCOMMERCIALCHART_END_NAMESPACE
             #endif // BAR_H

src/barchart/barchartitem.cpp +29 -31

             #include "barchartitem_p.h"
             #include "bar_p.h"
             #include "barvalue_p.h"
             #include "qbarset.h"
             #include "qbarseries.h"
             #include "qchart.h"
             #include "qchartaxis.h"
             #include "qchartaxiscategories.h"
             #include "chartpresenter_p.h"
             #include "chartanimator_p.h"
             #include "chartdataset_p.h"
             #include <QDebug>
             #include <QToolTip>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             BarChartItem::BarChartItem(QBarSeries *series, ChartPresenter *presenter) :
                 ChartItem(presenter),
                 mLayoutSet(false),
                 mSeries(series)
             {
-                connect(series,SIGNAL(showToolTip(QPoint,QString)),this,SLOT(showToolTip(QPoint,QString)));
+                connect(series, SIGNAL(showToolTip(QPoint,QString)), this, SLOT(showToolTip(QPoint,QString)));
                 connect(series, SIGNAL(updatedBars()), this, SLOT(handleLayoutChanged()));
             //TODO:  connect(series,SIGNAL("position or size has changed"), this, SLOT(handleLayoutChanged()));
                 connect(series, SIGNAL(restructuredBar(int)), this, SLOT(handleModelChanged(int)));
                 setZValue(ChartPresenter::BarSeriesZValue);
                 dataChanged();
             }
             BarChartItem::~BarChartItem()
             {
                 disconnect(this,SLOT(showToolTip(QPoint,QString)));
             }
             void BarChartItem::paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget)
             {
                 if (!mLayoutSet) {
                     qDebug() << "BarChartItem::paint called without layout set. Aborting.";
                     return;
                 }
-                foreach(QGraphicsItem* i, childItems()) {
+                foreach(QGraphicsItem* i, childItems())
                     i->paint(painter,option,widget);
             }
             QRectF BarChartItem::boundingRect() const
             {
                 return m_rect;
             }
             void BarChartItem::dataChanged()
             {
                 // TODO: performance optimizations. Do we really need to delete and create items every time data is changed or can we reuse them?
                 // Delete old bars
-                foreach (QGraphicsItem* item, childItems()) {
+                foreach (QGraphicsItem *item, childItems())
                     delete item;
                 mBars.clear();
                 mFloatingValues.clear();
                 mLayout.clear();
                 // Create new graphic items for bars
-                for (int c=0; c<mSeries->categoryCount(); c++) {
+                for (int c = 0; c < mSeries->categoryCount(); c++) {
                     QString category = mSeries->categoryName(c);
-                    for (int s=0; s<mSeries->barsetCount(); s++) {
+                    for (int s = 0; s < mSeries->barsetCount(); s++) {
                         QBarSet *set = mSeries->barsetAt(s);
                         Bar *bar = new Bar(category,this);
                         childItems().append(bar);
                         mBars.append(bar);
-                        connect(bar,SIGNAL(clicked(QString)),set,SIGNAL(clicked(QString)));
+                        connect(bar, SIGNAL(clicked(QString)), set, SIGNAL(clicked(QString)));
-                        connect(bar,SIGNAL(rightClicked(QString)),set,SIGNAL(rightClicked(QString)));
+                        connect(bar, SIGNAL(rightClicked(QString)), set, SIGNAL(rightClicked(QString)));
-                        connect(bar,SIGNAL(hoverEntered(QPoint)),set,SLOT(barHoverEnterEvent(QPoint)));
+                        connect(bar, SIGNAL(hoverEntered(QPoint)), set, SLOT(barHoverEnterEvent(QPoint)));
-                        connect(bar,SIGNAL(hoverLeaved()),set,SLOT(barHoverLeaveEvent()));
+                        connect(bar, SIGNAL(hoverLeaved()), set, SLOT(barHoverLeaveEvent()));
-                        mLayout.append(QRectF(0,0,0,0));
+                        mLayout.append(QRectF(0, 0, 0, 0));
                     }
                 }
                 // Create floating values
-                for (int category=0; category<mSeries->categoryCount(); category++) {
+                for (int category = 0; category < mSeries->categoryCount(); category++) {
-                    for (int s=0; s<mSeries->barsetCount(); s++) {
+                    for (int s = 0; s < mSeries->barsetCount(); s++) {
                         QBarSet *set = mSeries->barsetAt(s);
                         BarValue *value = new BarValue(*set, this);
                         childItems().append(value);
                         mFloatingValues.append(value);
-                        connect(set,SIGNAL(toggleFloatingValues()),value,SLOT(toggleVisible()));
+                        connect(set, SIGNAL(toggleFloatingValues()), value, SLOT(toggleVisible()));
                     }
                 }
             }
             QVector<QRectF> BarChartItem::calculateLayout()
             {
                 QVector<QRectF> layout;
                 // Use temporary qreals for accurancy (we might get some compiler warnings... :)
                 qreal categoryCount = mSeries->categoryCount();
                 qreal setCount = mSeries->barsetCount();
                 qreal width = geometry().width();
                 qreal height = geometry().height();
                 qreal max = mSeries->max();
                 // Domain:
                 if (mDomainMaxY > max) {
                     max = mDomainMaxY;
                 }
-                qreal scale = (height/max);
+                qreal scale = (height / max);
-                qreal categoryWidth = width/categoryCount;
+                qreal categoryWidth = width / categoryCount;
                 qreal barWidth = categoryWidth / (setCount+1);
                 int itemIndex(0);
-                for (int category=0; category < categoryCount; category++) {
+                for (int category = 0; category < categoryCount; category++) {
-                    qreal xPos = categoryWidth * category + barWidth/2;
+                    qreal xPos = categoryWidth * category + barWidth / 2;
                     qreal yPos = height;
                     for (int set = 0; set < setCount; set++) {
-                        qreal barHeight = mSeries->valueAt(set,category) * scale;
+                        qreal barHeight = mSeries->valueAt(set, category) * scale;
                         Bar* bar = mBars.at(itemIndex);
-                        QRectF rect(xPos,yPos-barHeight,barWidth,barHeight);
+                        QRectF rect(xPos, yPos - barHeight, barWidth, barHeight);
                         layout.append(rect);
                         bar->setPen(mSeries->barsetAt(set)->pen());
                         bar->setBrush(mSeries->barsetAt(set)->brush());
                         itemIndex++;
                         xPos += barWidth;
                     }
                 }
                 // Position floating values
                 itemIndex = 0;
-                for (int category=0; category < mSeries->categoryCount(); category++) {
+                for (int category = 0; category < mSeries->categoryCount(); category++) {
                     qreal xPos = categoryWidth * category  + barWidth;
                     qreal yPos = height;
                     for (int set=0; set < mSeries->barsetCount(); set++) {
-                        qreal barHeight = mSeries->valueAt(set,category) * scale;
+                        qreal barHeight = mSeries->valueAt(set, category) * scale;
                         BarValue* value = mFloatingValues.at(itemIndex);
                         QBarSet* barSet = mSeries->barsetAt(set);
-                        value->resize(100,50);  // TODO: proper layout for this.
+                        value->resize(100, 50);  // TODO: proper layout for this.
-                        value->setPos(xPos, yPos-barHeight/2);
+                        value->setPos(xPos, yPos-barHeight / 2);
                         value->setPen(barSet->floatingValuePen());
                         if (mSeries->valueAt(set,category) != 0) {
-                            value->setValueString(QString::number(mSeries->valueAt(set,category)));
+                            value->setValueString(QString::number(mSeries->valueAt(set, category)));
                         } else {
                             value->setValueString(QString(""));
                         }
                         itemIndex++;
                         xPos += barWidth;
                     }
                 }
                 return layout;
             }
             void BarChartItem::applyLayout(const QVector<QRectF> &layout)
             {
                 if (animator())
                     animator()->updateLayout(this, mLayout, layout);
                 else
                     setLayout(layout);
             }
             void BarChartItem::setLayout(const QVector<QRectF> &layout)
             {
                 mLayout = layout;
-                for (int i=0; i<mBars.count(); i++) {
+                for (int i=0; i < mBars.count(); i++)
                     mBars.at(i)->setRect(layout.at(i));
                 update();
             }
             //handlers
             void BarChartItem::handleModelChanged(int index)
             {
                 Q_UNUSED(index)
                 dataChanged();
             }
             void BarChartItem::handleDomainChanged(qreal minX, qreal maxX, qreal minY, qreal maxY)
             {
                 mDomainMinX = minX;
                 mDomainMaxX = maxX;
                 mDomainMinY = minY;
                 mDomainMaxY = maxY;
                 handleLayoutChanged();
             }
-            void BarChartItem::handleGeometryChanged(const QRectF& rect)
+            void BarChartItem::handleGeometryChanged(const QRectF &rect)
             {
-                m_rect=rect;
+                m_rect = rect;
                 handleLayoutChanged();
                 mLayoutSet = true;
                 setPos(rect.topLeft());
             }
             void BarChartItem::handleLayoutChanged()
             {
                 QVector<QRectF> layout = calculateLayout();
                 applyLayout(layout);
                 update();
             }
             void BarChartItem::showToolTip(QPoint pos, QString tip)
             {
                 // TODO: cool tooltip instead of default
-                QToolTip::showText(pos,tip);
+                QToolTip::showText(pos, tip);
             }
             #include "moc_barchartitem_p.cpp"
             QTCOMMERCIALCHART_END_NAMESPACE

src/barchart/barchartitem_p.h +4 -4

             #ifndef BARCHARTITEM_H
             #define BARCHARTITEM_H
             #include "chartitem_p.h"
             #include "qbarseries.h"
             #include <QPen>
             #include <QBrush>
             #include <QGraphicsItem>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             class Bar;
             class BarValue;
             class QChartAxisCategories;
             class QChart;
             //typedef QVector<QRectF> BarLayout;
             class BarChartItem : public ChartItem
             {
                 Q_OBJECT
             public:
                 BarChartItem(QBarSeries *series, ChartPresenter *presenter);
                 virtual ~BarChartItem();
             // Common implemantation of different presenters. Not to be instantiated.
             // TODO: combine this with BarPresenter and derive other presenters from it?
             public:
                 // From QGraphicsItem
                 void paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget);
                 QRectF boundingRect() const;
                 // TODO: Consider the domain for layoutChanged. May be use case, may not be. If it is, then the derived classes need to implement it
                 virtual void dataChanged();     // data of series has changed -> need to recalculate bar sizes
             private slots:
             public:
                 virtual QVector<QRectF> calculateLayout();
                 void applyLayout(const QVector<QRectF> &layout);
                 void setLayout(const QVector<QRectF> &layout);
                 void updateLayout(const QVector<QRectF> &layout);
                 QRectF geometry() const { return m_rect;}
             public slots:
                 void handleModelChanged(int index);
                 void handleDomainChanged(qreal minX, qreal maxX, qreal minY, qreal maxY);
-                void handleGeometryChanged(const QRectF& size);
+                void handleGeometryChanged(const QRectF &size);
                 void handleLayoutChanged();
                 // Internal slots
                 void showToolTip(QPoint pos, QString tip);      // shows tooltip (if enabled)
             protected:
                 // TODO: consider these.
                 qreal mDomainMinX;
                 qreal mDomainMaxX;
                 qreal mDomainMinY;
                 qreal mDomainMaxY;
                 QRectF m_rect;
                 bool mLayoutSet;    // True, if component has been laid out.
                 QVector<QRectF> mLayout;
                 // Not owned.
-                QBarSeries* mSeries;
+                QBarSeries *mSeries;
-                QList<Bar*> mBars;
+                QList<Bar *> mBars;
-                QList<BarValue*> mFloatingValues;
+                QList<BarValue *> mFloatingValues;
             };
             QTCOMMERCIALCHART_END_NAMESPACE
             #endif // BARCHARTITEM_H

src/barchart/barchartmodel.cpp +10 -15

             #include <limits.h>
             #include <QVector>
             #include <QDebug>
             #include "barchartmodel_p.h"
             #include "qbarset.h"
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             BarChartModel::BarChartModel(QStringList categories, QObject *parent) :
                 QObject(parent)
                 ,mCategory(categories)
             {
             }
             QStringList BarChartModel::category()
             {
                 return mCategory;
             }
             void BarChartModel::addBarSet(QBarSet *set)
             {
                 mDataModel.append(set);
             }
             void BarChartModel::removeBarSet(QBarSet *set)
             {
                 if (mDataModel.contains(set)) {
                     mDataModel.removeOne(set);
                 }
             }
             void BarChartModel::insertBarSet(int i, QBarSet *set)
             {
                 mDataModel.insert(i, set);
             }
             void BarChartModel::insertCategory(int i, QString category)
             {
                 mCategory.insert(i, category);
             }
             void BarChartModel::removeCategory(int i)
             {
                 mCategory.removeAt(i);
             }
             QBarSet* BarChartModel::setAt(int index)
             {
                 return mDataModel.at(index);
             }
             QList<QBarSet*> BarChartModel::barSets()
             {
                 return mDataModel;
             }
             int BarChartModel::barsetCount()
             {
                 return mDataModel.count();
             }
             int BarChartModel::categoryCount()
             {
                 return mCategory.count();
             }
             qreal BarChartModel::min()
             {
                 Q_ASSERT(mDataModel.count() > 0);
                 // TODO: make min and max members and update them when data changes.
                 // This is slower since they are checked every time, even if data is same since previous call.
                 qreal min = INT_MAX;
-                for (int i=0; i <mDataModel.count(); i++) {
+                for (int i = 0; i < mDataModel.count(); i++) {
                     int itemCount = mDataModel.at(i)->count();
-                    for (int j=0; j<itemCount; j++) {
+                    for (int j = 0; j < itemCount; j++) {
                         qreal temp = mDataModel.at(i)->valueAt(j);
-                        if (temp < min) {
+                        if (temp < min)
                             min = temp;
                     }
                 }
                 return min;
             }
             qreal BarChartModel::max()
             {
                 Q_ASSERT(mDataModel.count() > 0);
                 // TODO: make min and max members and update them when data changes.
                 // This is slower since they are checked every time, even if data is same since previous call.
                 qreal max = INT_MIN;
-                for (int i=0; i <mDataModel.count(); i++) {
+                for (int i = 0; i < mDataModel.count(); i++) {
                     int itemCount = mDataModel.at(i)->count();
-                    for (int j=0; j<itemCount; j++) {
+                    for (int j = 0; j < itemCount; j++) {
                         qreal temp = mDataModel.at(i)->valueAt(j);
-                        if (temp > max) {
+                        if (temp > max)
                             max = temp;
                     }
                 }
                 return max;
             }
             qreal BarChartModel::valueAt(int set, int category)
             {
                 if ((set < 0) || (set >= mDataModel.count())) {
                     // No set, no value.
                     return 0;
                 } else if ((category < 0) || (category >= mDataModel.at(set)->count())) {
                     // No category, no value.
                     return 0;
                 }
                 return mDataModel.at(set)->valueAt(category);
             }
             qreal BarChartModel::percentageAt(int set, int category)
             {
                 if ((set < 0) || (set >= mDataModel.count())) {
                     // No set, no value.
                     return 0;
                 } else if ((category < 0) || (category >= mDataModel.at(set)->count())) {
                     // No category, no value.
                     return 0;
                 }
                 qreal value = mDataModel.at(set)->valueAt(category);
                 qreal total = categorySum(category);
-                if (0 == total) {
+                if (0 == total)
                     return 100.0;
                 return value / total;
             }
             qreal BarChartModel::categorySum(int category)
             {
                 qreal sum(0);
                 int count = mDataModel.count(); // Count sets
                 for (int set = 0; set < count; set++) {
-                    if (category < mDataModel.at(set)->count()) {
+                    if (category < mDataModel.at(set)->count())
                         sum += mDataModel.at(set)->valueAt(category);
                 }
                 return sum;
             }
             qreal BarChartModel::maxCategorySum()
             {
                 qreal max = INT_MIN;
                 int count = categoryCount();
-                for (int col=0; col<count; col++) {
+                for (int col = 0; col < count; col++) {
                     qreal sum = categorySum(col);
-                    if (sum > max) {
+                    if (sum > max)
                         max = sum;
                 }
                 return max;
             }
             QString BarChartModel::categoryName(int category)
             {
                 return mCategory.at(category);
             }
             #include "moc_barchartmodel_p.cpp"
             QTCOMMERCIALCHART_END_NAMESPACE

src/barchart/barchartmodel_p.h +2 -4

             #ifndef BARCHARTMODEL_H
             #define BARCHARTMODEL_H
             #include <QObject>
             #include <QStringList>
             #include "qchartglobal.h"
             #include <qseries.h>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             // Model for bar chart. Internal class.
             // TODO: Implement as QAbstractItemModel?
             class QBarSet;
             class BarChartModel : public QObject //, public QAbstractItemModel
             {
                 Q_OBJECT
             public:
                 explicit BarChartModel(QStringList categories, QObject *parent = 0);
                 QStringList category();
                 void addBarSet(QBarSet *set);
                 void removeBarSet(QBarSet *set);
                 void insertBarSet(int i, QBarSet *set);
                 void insertCategory(int i, QString category);
                 void removeCategory(int i);
                 QBarSet *setAt(int index);
-                QList<QBarSet*> barSets();
+                QList<QBarSet *> barSets();
                 int barsetCount();          // Number of sets in model
                 int categoryCount();        // Number of categories
                 qreal max();                // Maximum value of all sets
                 qreal min();                // Minimum value of all sets
                 qreal valueAt(int set, int category);
                 qreal percentageAt(int set, int category);
                 qreal categorySum(int category);
                 qreal maxCategorySum();     // returns maximum sum of sets in all categories.
                 QString categoryName(int category);
             signals:
                 void modelUpdated();
             public slots:
             private:
-                QList<QBarSet*> mDataModel;
+                QList<QBarSet *> mDataModel;
                 QStringList mCategory;
                 int mCurrentSet;
             };
             QTCOMMERCIALCHART_END_NAMESPACE
             #endif // BARCHARTMODEL_H

src/barchart/barvalue.cpp +1 -1

             #include "barvalue_p.h"
             #include <QPainter>
             #include <QPen>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             BarValue::BarValue(QBarSet &set, QGraphicsItem *parent)
                 : QGraphicsObject(parent),
                 mBarSet(set),
                 mXpos(0),
                 mYpos(0),
                 mWidth(0),
                 mHeight(0)
             {
                 setVisible(false);
             }
             void BarValue::setValueString(QString str)
             {
                 mValueString = str;
             }
             QString BarValue::valueString()
             {
                 return mValueString;
             }
             void BarValue::setPen(const QPen pen)
             {
                 mPen = pen;
             }
             QPen BarValue::pen() const
             {
                 return mPen;
             }
             void BarValue::resize(qreal w, qreal h)
             {
                 mWidth = w;
                 mHeight = h;
             }
             void BarValue::setPos(qreal x, qreal y)
             {
                 mXpos = x;
                 mYpos = y;
             }
             void BarValue::paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget)
             {
                 Q_UNUSED(option)
                 Q_UNUSED(widget)
                 if (isVisible()) {
                     painter->setPen(mPen);
-                    painter->drawText(boundingRect(),mValueString);
+                    painter->drawText(boundingRect(), mValueString);
                 }
             }
             QRectF BarValue::boundingRect() const
             {
                 QRectF r(mXpos, mYpos, mWidth, mHeight);
                 return r;
             }
             void BarValue::toggleVisible()
             {
                 setVisible(!isVisible());
             }
             #include "moc_barvalue_p.cpp"
             QTCOMMERCIALCHART_END_NAMESPACE

src/barchart/barvalue_p.h +1 -1

             #ifndef BARVALUE_P_H
             #define BARVALUE_P_H
             #include "qchartglobal.h"
             #include <QGraphicsObject>
             #include <QPen>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             class QBarSet;
             // Visual class for floating bar values
             // By default these are not visible.
             class BarValue : public QGraphicsObject
             {
                 Q_OBJECT
             public:
                 BarValue(QBarSet &set, QGraphicsItem *parent = 0);
                 void setValueString(QString str);
                 QString valueString();
                 void setPen(const QPen pen);
                 QPen pen() const;
                 void resize(qreal w, qreal h);
                 void setPos(qreal x, qreal y);
                 // From QGraphicsItem
                 void paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget);
                 QRectF boundingRect() const;
             public Q_SLOTS:
                 void toggleVisible();
             private:
-                QBarSet& mBarSet;
+                QBarSet &mBarSet;
                 QPen mPen;
                 QString mValueString;
                 qreal mXpos;
                 qreal mYpos;
                 qreal mWidth;
                 qreal mHeight;
             };
             QTCOMMERCIALCHART_END_NAMESPACE
             #endif // BARVALUE_P_H

src/barchart/percentbarchartitem.cpp +7 -7

             #include "percentbarchartitem_p.h"
             #include "bar_p.h"
             #include "barvalue_p.h"
             #include "qbarset.h"
             #include <QDebug>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
-            PercentBarChartItem::PercentBarChartItem(QBarSeries *series, ChartPresenter* presenter) :
+            PercentBarChartItem::PercentBarChartItem(QBarSeries *series, ChartPresenter *presenter) :
                 BarChartItem(series, presenter)
             {
             }
             QVector<QRectF> PercentBarChartItem::calculateLayout()
             {
                 QVector<QRectF> layout;
                 // Use temporary qreals for accurancy (we might get some compiler warnings... :)
                 qreal width = geometry().width();
                 qreal height = geometry().height();
                 qreal categoryCount = mSeries->categoryCount();
                 qreal barWidth = width / (mSeries->categoryCount() * 2);
-                qreal xStep = width/categoryCount;
+                qreal xStep = width / categoryCount;
-                qreal xPos =  xStep/2 - barWidth / 2;
+                qreal xPos =  xStep / 2 - barWidth / 2;
                 int itemIndex(0);
-                for (int category = 0; category < categoryCount ; category++) {
+                for (int category = 0; category < categoryCount; category++) {
                     qreal colSum = mSeries->categorySum(category);
                     qreal scale = (height / colSum);
                     qreal yPos = height;
                     for (int set=0; set < mSeries->barsetCount(); set++) {
                         qreal barHeight = mSeries->valueAt(set, category) * scale;
                         Bar* bar = mBars.at(itemIndex);
                         bar->setPen(mSeries->barsetAt(set)->pen());
                         bar->setBrush(mSeries->barsetAt(set)->brush());
-                        QRectF rect(xPos, yPos-barHeight,barWidth, barHeight);
+                        QRectF rect(xPos, yPos-barHeight, barWidth, barHeight);
                         layout.append(rect);
                         itemIndex++;
                         yPos -= barHeight;
                     }
                     xPos += xStep;
                 }
                 // Position floating values
                 itemIndex = 0;
                 xPos = (width/categoryCount);
                 for (int category=0; category < mSeries->categoryCount(); category++) {
                     qreal yPos = height;
                     qreal colSum = mSeries->categorySum(category);
                     qreal scale = (height / colSum);
                     for (int set=0; set < mSeries->barsetCount(); set++) {
                         qreal barHeight = mSeries->valueAt(set,category) * scale;
                         BarValue* value = mFloatingValues.at(itemIndex);
                         QBarSet* barSet = mSeries->barsetAt(set);
-                        value->resize(100,50);  // TODO: proper layout for this.
+                        value->resize(100, 50);  // TODO: proper layout for this.
-                        value->setPos(xPos, yPos-barHeight/2);
+                        value->setPos(xPos, yPos-barHeight / 2);
                         value->setPen(barSet->floatingValuePen());
                         if (mSeries->valueAt(set,category) != 0) {
                             int p = mSeries->percentageAt(set,category) * 100;
                             QString vString(QString::number(p));
                             vString.truncate(3);
                             vString.append("%");
                             value->setValueString(vString);
                         } else {
                             value->setValueString(QString(""));
                         }
                         itemIndex++;
                         yPos -= barHeight;
                     }
                     xPos += xStep;
                 }
                 return layout;
             }
             #include "moc_percentbarchartitem_p.cpp"
             QTCOMMERCIALCHART_END_NAMESPACE

src/barchart/qbarseries.cpp +38 -41

             #include <QDebug>
             #include "qbarseries.h"
             #include "qbarset.h"
             #include "barchartmodel_p.h"
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             /*!
                 \class QBarSeries
                 \brief part of QtCommercial chart API.
                 QBarSeries represents a series of data shown as bars. One QBarSeries can contain multible
                 QBarSet data sets. QBarSeries groups the data from sets to categories, which are defined
                 by QStringList.
                 \mainclass
                 \sa QBarSet, QStackedBarSeries, QPercentBarSeries
             */
             /*!
                 \fn virtual QSeriesType QBarSeries::type() const
                 \brief Returns type of series.
                 \sa QSeries, QSeriesType
             */
             /*!
                 \fn void QBarSeries::showToolTip(QPoint pos, QString tip)
                 \brief \internal \a pos \a tip
             */
             /*!
                 Constructs empty QBarSeries. Parameter \a categories defines the categories for chart.
                 QBarSeries is QObject which is a child of a \a parent.
             */
             QBarSeries::QBarSeries(QBarCategories categories, QObject *parent)
                 : QSeries(parent)
                 ,mModel(new BarChartModel(categories, this))
             {
                 m_model = NULL;
                 m_mapCategories = -1;
                 m_mapBarBottom = -1;
                 m_mapBarTop = -1;
                 m_mapFirst = 0;
                 m_mapCount = 0;
                 m_mapOrientation = Qt::Vertical;
             }
             /*!
                 Adds a set of bars to series. Takes ownership of \a set.
                 Connects the clicked(QString) and rightClicked(QString) signals
                 of \a set to this series
             */
             void QBarSeries::addBarSet(QBarSet *set)
             {
                 mModel->addBarSet(set);
-                connect(set,SIGNAL(clicked(QString)),this,SLOT(barsetClicked(QString)));
+                connect(set, SIGNAL(clicked(QString)), this, SLOT(barsetClicked(QString)));
-                connect(set,SIGNAL(rightClicked(QString)),this,SLOT(barsetRightClicked(QString)));
+                connect(set, SIGNAL(rightClicked(QString)), this, SLOT(barsetRightClicked(QString)));
                 connect(set, SIGNAL(valueChanged()), this, SLOT(barsetChanged()));
                 emit updatedBars();
             }
             /*!
                 Removes a set of bars from series. Releases ownership of \a set. Doesnt delete \a set.
                 Disconnects the clicked(QString) and rightClicked(QString) signals
                 of \a set from this series
             */
             void QBarSeries::removeBarSet(QBarSet *set)
             {
-                disconnect(set,SIGNAL(clicked(QString)),this,SLOT(barsetClicked(QString)));
+                disconnect(set, SIGNAL(clicked(QString)), this, SLOT(barsetClicked(QString)));
-                disconnect(set,SIGNAL(rightClicked(QString)),this,SLOT(barsetRightClicked(QString)));
+                disconnect(set, SIGNAL(rightClicked(QString)), this, SLOT(barsetRightClicked(QString)));
                 mModel->removeBarSet(set);
                 emit updatedBars();
             }
             void QBarSeries::insertBarSet(int i, QBarSet *set)
             {
                 mModel->insertBarSet(i, set);
                 //    emit barsetChanged();
             }
             void QBarSeries::insertCategory(int i, QString category)
             {
                 mModel->insertCategory(i, category);
             }
             void QBarSeries::removeCategory(int i)
             {
                 mModel->removeCategory(i);
             }
             /*!
                 Returns number of sets in series.
             */
             int QBarSeries::barsetCount()
             {
                 //    if(m_model)
                 //        return m_mapBarTop - m_mapBarBottom;
                 //    else
                 return mModel->barsetCount();
             }
             /*!
                 Returns number of categories in series
             */
             int QBarSeries::categoryCount()
             {
                 return mModel->categoryCount();
             }
             /*!
                 Returns a list of sets in series. Keeps ownership of sets.
              */
             QList<QBarSet*> QBarSeries::barSets()
             {
                 return mModel->barSets();
             }
             /*!
                 \internal \a index
             */
             QBarSet* QBarSeries::barsetAt(int index)
             {
                 return mModel->setAt(index);
             }
             /*!
                 \internal \a category
             */
             QString QBarSeries::categoryName(int category)
             {
                 return mModel->categoryName(category);
             }
             /*!
                 Enables or disables tooltip depending on parameter \a enabled.
                 Tooltip shows the name of set, when mouse is hovering on top of bar.
                 Calling without parameter \a enabled, enables the tooltip
             */
             void QBarSeries::setToolTipEnabled(bool enabled)
             {
                 // TODO: what if we add sets after call to this function? Those sets won't have tooltip enabled.
                 if (enabled) {
                     for (int i=0; i<mModel->barsetCount(); i++) {
                         QBarSet *set = mModel->setAt(i);
-                        connect(set,SIGNAL(showToolTip(QPoint,QString)),this,SIGNAL(showToolTip(QPoint,QString)));
+                        connect(set, SIGNAL(showToolTip(QPoint,QString)), this, SIGNAL(showToolTip(QPoint,QString)));
                     }
                 } else {
                     for (int i=0; i<mModel->barsetCount(); i++) {
                         QBarSet *set = mModel->setAt(i);
-                        disconnect(set,SIGNAL(showToolTip(QPoint,QString)),this,SIGNAL(showToolTip(QPoint,QString)));
+                        disconnect(set, SIGNAL(showToolTip(QPoint,QString)), this, SIGNAL(showToolTip(QPoint,QString)));
                     }
                 }
             }
             /*!
                 \internal \a category
             */
             void QBarSeries::barsetClicked(QString category)
             {
                 emit clicked(qobject_cast<QBarSet*>(sender()), category);
             }
             /*!
                 \internal \a category
             */
             void QBarSeries::barsetRightClicked(QString category)
             {
                 emit rightClicked(qobject_cast<QBarSet*>(sender()), category);
             }
             /*!
                 \internal
             */
             qreal QBarSeries::min()
             {
                 return mModel->min();
             }
             /*!
                 \internal
             */
             qreal QBarSeries::max()
             {
                 return mModel->max();
             }
             /*!
                 \internal \a set \a category
             */
             qreal QBarSeries::valueAt(int set, int category)
             {
-                return mModel->valueAt(set,category);
+                return mModel->valueAt(set, category);
             }
             /*!
                 \internal \a set \a category
             */
             qreal QBarSeries::percentageAt(int set, int category)
             {
-                return mModel->percentageAt(set,category);
+                return mModel->percentageAt(set, category);
             }
             /*!
                 \internal \a category
             */
             qreal QBarSeries::categorySum(int category)
             {
                 return mModel->categorySum(category);
             }
             /*!
                 \internal
             */
             qreal QBarSeries::maxCategorySum()
             {
                 return mModel->maxCategorySum();
             }
             /*!
                 \internal
             */
             BarChartModel& QBarSeries::model()
             {
                 return *mModel;
             }
-            bool QBarSeries::setModel(QAbstractItemModel* model)
+            bool QBarSeries::setModel(QAbstractItemModel *model)
             {
                 // disconnect signals from old model
                 if(m_model)
                 {
                     disconnect(m_model, 0, this, 0);
                     m_mapCategories = -1;
                     m_mapBarBottom = -1;
                     m_mapBarTop = -1;
                     m_mapFirst = 0;
                     m_mapCount = 0;
                     m_mapOrientation = Qt::Vertical;
                 }
                 // set new model
                 if(model)
                 {
                     m_model = model;
                     return true;
                 }
                 else
                 {
                     m_model = NULL;
                     return false;
                 }
             }
             // TODO
             void QBarSeries::setModelMapping(int categories, int bottomBoundry, int topBoundry, Qt::Orientation orientation)
             {
                 if (m_model == NULL)
                     return;
                 m_mapCategories = categories;
                 m_mapBarBottom = bottomBoundry;
                 m_mapBarTop = topBoundry;
             //    m_mapFirst = 1;
                 m_mapOrientation = orientation;
                 // connect the signals
-                if (m_mapOrientation == Qt::Vertical)
+                if (m_mapOrientation == Qt::Vertical) {
                     m_mapCount = m_model->rowCount() - m_mapFirst;
-                    connect(m_model,SIGNAL(dataChanged(QModelIndex,QModelIndex)), this, SLOT(modelUpdated(QModelIndex, QModelIndex)));
+                    connect(m_model,SIGNAL(dataChanged(QModelIndex,QModelIndex)),
-                    connect(m_model,SIGNAL(rowsInserted(QModelIndex, int, int)), this, SLOT(modelDataAdded(QModelIndex,int,int)));
+                            this, SLOT(modelUpdated(QModelIndex, QModelIndex)));
-                    connect(m_model, SIGNAL(rowsRemoved(QModelIndex, int, int)), this, SLOT(modelDataRemoved(QModelIndex,int,int)));
+                    connect(m_model,SIGNAL(rowsInserted(QModelIndex, int, int)),
+                            this, SLOT(modelDataAdded(QModelIndex,int,int)));
-                else
+                    connect(m_model, SIGNAL(rowsRemoved(QModelIndex, int, int)),
+                            this, SLOT(modelDataRemoved(QModelIndex,int,int)));
+                } else {
                     m_mapCount = m_model->columnCount() - m_mapFirst;
-                    connect(m_model,SIGNAL(dataChanged(QModelIndex,QModelIndex)), this, SLOT(modelUpdated(QModelIndex, QModelIndex)));
+                    connect(m_model,SIGNAL(dataChanged(QModelIndex,QModelIndex)),
-                    connect(m_model,SIGNAL(columnsInserted(QModelIndex, int, int)), this, SLOT(modelDataAdded(QModelIndex,int,int)));
+                            this, SLOT(modelUpdated(QModelIndex, QModelIndex)));
-                    connect(m_model, SIGNAL(columnsRemoved(QModelIndex, int, int)), this, SLOT(modelDataRemoved(QModelIndex,int,int)));
+                    connect(m_model,SIGNAL(columnsInserted(QModelIndex, int, int)),
+                            this, SLOT(modelDataAdded(QModelIndex,int,int)));
+                    connect(m_model, SIGNAL(columnsRemoved(QModelIndex, int, int)),
+                            this, SLOT(modelDataRemoved(QModelIndex,int,int)));
                 }
                 // create the initial bars
                 delete mModel;
-                if (m_mapOrientation == Qt::Vertical)
+                if (m_mapOrientation == Qt::Vertical) {
                     QStringList categories;
                     for (int k = m_mapFirst; k < m_mapFirst + m_mapCount; k++)
                         categories << m_model->data(m_model->index(k, m_mapCategories), Qt::DisplayRole).toString();
                     mModel = new BarChartModel(categories, this);
-                    for (int i = m_mapBarBottom; i <= m_mapBarTop; i++)
+                    for (int i = m_mapBarBottom; i <= m_mapBarTop; i++) {
                         QBarSet* barSet = new QBarSet(QString("Column: %1").arg(i + 1));
                         for(int m = m_mapFirst; m < m_mapFirst + m_mapCount; m++)
                             *barSet << m_model->data(m_model->index(m, i), Qt::DisplayRole).toDouble();
                         addBarSet(barSet);
                     }
+                } else {
-                else
                     QStringList categories;
                     for (int k = m_mapFirst; k < m_mapFirst + m_mapCount; k++)
                         categories << m_model->data(m_model->index(m_mapCategories, k), Qt::DisplayRole).toString();
                     mModel = new BarChartModel(categories, this);
-                    for (int i = m_mapBarBottom; i <= m_mapBarTop; i++)
+                    for (int i = m_mapBarBottom; i <= m_mapBarTop; i++) {
                         QBarSet* barSet = new QBarSet(QString("Row: %1").arg(i + 1));
                         for(int m = m_mapFirst; m < m_mapFirst + m_mapCount; m++)
                             *barSet << m_model->data(m_model->index(i, m), Qt::DisplayRole).toDouble();
                         addBarSet(barSet);
                     }
                 }
             }
             void QBarSeries::setModelMappingShift(int first, int count)
             {
                 m_mapFirst = first;
                 m_mapCount = count;
             }
             void QBarSeries::modelUpdated(QModelIndex topLeft, QModelIndex bottomRight)
             {
                 Q_UNUSED(bottomRight)
                 if (m_mapOrientation == Qt::Vertical)
                 {
                     // model update is relevant to BarSeries if the change was made to the part of the model that was mapped to BarSeries
                     if (topLeft.column() >= m_mapBarBottom && topLeft.column() <= m_mapBarTop && topLeft.row() >= m_mapFirst && topLeft.row() < m_mapFirst + m_mapCount)
                         barsetAt(topLeft.column() - m_mapBarBottom)->setValue(topLeft.row() - m_mapFirst, m_model->data(topLeft, Qt::DisplayRole).toDouble());
                 }
                 else
                 {
                     // model update is relevant to BarSeries if the change was made to the part of the model that was mapped to BarSeries
                     if (topLeft.row() >= m_mapBarBottom && topLeft.row() <= m_mapBarTop && topLeft.column() >= m_mapFirst && topLeft.column() < m_mapFirst + m_mapCount)
                         barsetAt(topLeft.row() - m_mapBarBottom)->setValue(topLeft.column() - m_mapFirst, m_model->data(topLeft, Qt::DisplayRole).toDouble());
                 }
             }
             void QBarSeries::modelDataAdded(QModelIndex /*parent*/, int start, int /*end*/)
             {
-                if (m_mapOrientation == Qt::Vertical)
+                if (m_mapOrientation == Qt::Vertical) {
                     insertCategory(start - m_mapFirst, QString("Row: %1").arg(start + 1));
-                    for (int i = 0; i <= m_mapBarTop - m_mapBarBottom; i++)
+                    for (int i = 0; i <= m_mapBarTop - m_mapBarBottom; i++) {
                         barsetAt(i)->insertValue(start - m_mapFirst, m_model->data(m_model->index(start, i), Qt::DisplayRole).toDouble());
                     }
+                } else {
-                else
                     insertCategory(start - m_mapFirst, QString("Column: %1").arg(start + 1));
-                    for (int i = 0; i <= m_mapBarTop - m_mapBarBottom; i++)
+                    for (int i = 0; i <= m_mapBarTop - m_mapBarBottom; i++) {
                         barsetAt(i)->insertValue(start - m_mapFirst, m_model->data(m_model->index(i, start), Qt::DisplayRole).toDouble());
                     }
                 }
                 emit restructuredBar(1);
             }
-            void QBarSeries::modelDataRemoved(QModelIndex /*parent*/, int start, int /*end*/)
+            void QBarSeries::modelDataRemoved(QModelIndex parent, int start, int end)
             {
+                Q_UNUSED(parent)
+                Q_UNUSED(end)
                 removeCategory(start - m_mapFirst);
                 for (int i = 0; i <= m_mapBarTop - m_mapBarBottom; i++)
                 {
                     barsetAt(i)->removeValue(start - m_mapFirst);
                 }
                 emit restructuredBar(1);
             }
             void QBarSeries::barsetChanged()
             {
                 emit updatedBars();
             }
             QBarCategories QBarSeries::categories() const
             {
                 QBarCategories categories;
                 int count = mModel->categoryCount();
-                for (int i=1; i<=count; i++) {
+                for (int i=1; i <= count; i++) {
-                      categories.insert(i, mModel->categoryName(i-1));
+                      categories.insert(i, mModel->categoryName(i - 1));
                 }
                 return categories;
             }
             #include "moc_qbarseries.cpp"
             QTCOMMERCIALCHART_END_NAMESPACE

src/barchart/qbarseries.h +7 -8

             #ifndef BARSERIES_H
             #define BARSERIES_H
             #include <qseries.h>
             #include <QStringList>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             typedef QStringList QBarCategories;
             class QBarSet;
             class BarChartModel;
             class BarCategory;
             // Container for series
             class QTCOMMERCIALCHART_EXPORT QBarSeries : public QSeries
             {
                 Q_OBJECT
             public:
-                QBarSeries(QStringList categories, QObject* parent=0);
+                QBarSeries(QStringList categories, QObject *parent = 0);
                 virtual QSeriesType type() const { return QSeries::SeriesTypeBar; }
                 void addBarSet(QBarSet *set);               // Takes ownership of set
                 void removeBarSet(QBarSet *set);            // Releases ownership, doesn't delete set
                 void insertBarSet(int i, QBarSet *set);
                 void insertCategory(int i, QString category);
                 void removeCategory(int i);
                 int barsetCount();
                 int categoryCount();
                 QList<QBarSet*> barSets();
                 QBarCategories categories() const;
+                bool setModel(QAbstractItemModel *model);
-                bool setModel(QAbstractItemModel* model);
+                QAbstractItemModel *modelExt() { return m_model; }
-                QAbstractItemModel* modelExt() {return m_model;}
                 void setModelMapping(int categories, int bottomBoundry, int topBoundry, Qt::Orientation orientation = Qt::Vertical);
                 void setModelMappingShift(int first, int count);
             public:
                 // TODO: Functions below this are not part of api and will be moved
                 // to private implementation, when we start using it
                 // TODO: TO PIMPL --->
                 QBarSet* barsetAt(int index);
                 QString categoryName(int category);
                 qreal min();
                 qreal max();
                 qreal valueAt(int set, int category);
                 qreal percentageAt(int set, int category);
                 qreal categorySum(int category);
                 qreal maxCategorySum();
                 BarChartModel& model();
                 // <--- TO PIMPL
             signals:
                 //void changed(int index);
-                void clicked(QBarSet* barset, QString category);        // Up to user of api, what to do with these signals
+                void clicked(QBarSet *barset, QString category);        // Up to user of api, what to do with these signals
-                void rightClicked(QBarSet* barset, QString category);
+                void rightClicked(QBarSet *barset, QString category);
                 //
                 void updatedBars();
                 void restructuredBar(int);
                 // TODO: internal signals, these to private implementation.
                 // TODO: TO PIMPL --->
                 void showToolTip(QPoint pos, QString tip);
                 // <--- TO PIMPL
             public Q_SLOTS:
-                void setToolTipEnabled(bool enabled=true);           // enables tooltips
+                void setToolTipEnabled(bool enabled = true);           // enables tooltips
                 // TODO: TO PIMPL --->
                 void barsetClicked(QString category);
                 void barsetRightClicked(QString category);
                 // <--- TO PIMPL
             private Q_SLOTS:
                 // slots for updating bars when data in model changes
                 void modelUpdated(QModelIndex topLeft, QModelIndex bottomRight);
                 void modelDataAdded(QModelIndex parent, int start, int end);
                 void modelDataRemoved(QModelIndex parent, int start, int end);
                 void barsetChanged();
             protected:
-                BarChartModel* mModel;
+                BarChartModel *mModel;
             //    QAbstractItemModel* m_model;
                 int m_mapCategories;
                 int m_mapBarBottom;
                 int m_mapBarTop;
                 int m_mapFirst;
                 int m_mapCount;
                 Qt::Orientation m_mapOrientation;
             };
             QTCOMMERCIALCHART_END_NAMESPACE
             #endif // BARSERIES_H

src/barchart/qbarset.cpp +2 -1

             #include "qbarset.h"
             #include <QDebug>
             #include <QToolTip>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             /*!
                 \class QBarSet
                 \brief part of QtCommercial chart API.
                 QBarSet represents one set of bars. Set of bars contains one data value for each category.
                 First value of set is assumed to belong to first category, second to second category and so on.
                 If set has fewer values than there are categories, then the missing values are assumed to be
                 at the end of set. For missing values in middle of a set, numerical value of zero is used.
                 \mainclass
                 \sa QBarSeries, QStackedBarSeries, QPercentBarSeries
             */
             /*!
                 \fn void QBarSet::clicked(QString category)
                 \brief signals that set has been clicked
                 Parameter \a category describes on which category was clicked
             */
             /*!
                 \fn void QBarSet::rightClicked(QString category)
                 \brief signals that set has been clicked with right mouse button
                 Parameter \a category describes on which category was clicked
             */
             /*!
                 \fn void QBarSet::hoverEnter(QPoint pos)
                 \brief signals that mouse has entered over the set at position \a pos.
             */
             /*!
                 \fn void QBarSet::hoverLeave()
                 \brief signals that mouse has left from the set.
             */
             /*!
                 \fn void QBarSet::toggleFloatingValues()
                 \brief \internal
             */
             /*!
                 \fn void QBarSet::showToolTip(QPoint pos, QString tip)
                 \brief \internal \a pos \a tip
             */
             /*!
                 Constructs QBarSet with a name of \a name and with parent of \a parent
             */
             QBarSet::QBarSet(QString name, QObject *parent)
                 : QObject(parent)
                 ,mName(name)
             {
             }
             /*!
                 Sets new \a name for set.
             */
             void QBarSet::setName(QString name)
             {
                 mName = name;
             }
             /*!
                 Returns name of the set.
             */
             QString QBarSet::name()
             {
                 return mName;
             }
             /*!
                 Appends new value \a value to the end of set.
             */
             QBarSet& QBarSet::operator << (const qreal &value)
             {
                 mValues.append(value);
                 emit structureChanged();
                 return *this;
             }
             void QBarSet::insertValue(int i, qreal value)
             {
                 mValues.insert(i, value);
             }
             void QBarSet::removeValue(int i)
             {
                 mValues.removeAt(i);
             }
             /*!
                 Returns count of values in set.
             */
             int QBarSet::count()
             {
                 return mValues.count();
             }
             /*!
                 Returns value of set indexed by \a index
             */
             qreal QBarSet::valueAt(int index)
             {
                 return mValues.at(index);
             }
             /*!
                 Sets a new value \a value to set, indexed by \a index
             */
             void QBarSet::setValue(int index, qreal value)
             {
                 mValues.replace(index,value);
                 emit valueChanged();
             }
             /*!
                 Returns total sum of all values in barset.
             */
             qreal QBarSet::total()
             {
                 qreal total(0);
-                for (int i=0; i<mValues.count(); i++) {
+                for (int i=0; i < mValues.count(); i++) {
                     total += mValues.at(i);
                 }
                 return total;
             }
             /*!
                 Sets pen for set. Bars of this set are drawn using \a pen
             */
             void QBarSet::setPen(const QPen pen)
             {
                 mPen = pen;
                 emit valueChanged();
             }
             /*!
                 Returns pen of the set.
             */
             QPen QBarSet::pen() const
             {
                 return mPen;
             }
             /*!
                 Sets brush for the set. Bars of this set are drawn using \a brush
             */
             void QBarSet::setBrush(const QBrush brush)
             {
                 mBrush = brush;
                 emit valueChanged();
             }
             /*!
                 Returns brush of the set.
             */
             QBrush QBarSet::brush() const
             {
                 return mBrush;
             }
             /*!
                 Sets the pen for floating values that are drawn on top of this set
             */
             void QBarSet::setFloatingValuePen(const QPen pen)
             {
                 mFloatingValuePen = pen;
             }
             /*!
                 Returns the pen for floating values that are drawn on top of this set
             */
             QPen QBarSet::floatingValuePen() const
             {
                 return mFloatingValuePen;
             }
             /*!
                 \internal \a pos
             */
             void QBarSet::barHoverEnterEvent(QPoint pos)
             {
                 emit showToolTip(pos, mName);
                 emit hoverEnter(pos);
             }
             /*!
                 \internal
             */
             void QBarSet::barHoverLeaveEvent()
             {
                 // Emit signal to user of charts
                 emit hoverLeave();
             }
             #include "moc_qbarset.cpp"
             QTCOMMERCIALCHART_END_NAMESPACE

src/barchart/qbarset.h +1 -1

             #ifndef QBARSET_H
             #define QBARSET_H
             #include <qchartglobal.h>
             #include <QPen>
             #include <QBrush>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             class QTCOMMERCIALCHART_EXPORT QBarSet : public QObject
             {
                 Q_OBJECT
             public:
                 QBarSet(QString name, QObject *parent = 0);
                 void setName(QString name);
                 QString name();
                 QBarSet& operator << (const qreal &value);  // appends new value to set
                 void insertValue(int i, qreal value);
                 void removeValue(int i);
                 // TODO: remove indices eventually. Use as internal?
                 int count();                                // count of values in set
                 qreal valueAt(int index);                   // for modifying individual values
                 void setValue(int index, qreal value);      // setter for individual value
                 qreal total();                              // total values in the set
                 // TODO:
                 //qreal value(QString category);
                 //void setValue(QString category, qreal value);
                 void setPen(const QPen pen);
                 QPen pen() const;
                 void setBrush(const QBrush brush);
                 QBrush brush() const;
                 void setFloatingValuePen(const QPen pen);
                 QPen floatingValuePen() const;
             Q_SIGNALS:
                 void clicked(QString category);                         // Clicked and hover signals exposed to user
                 void rightClicked(QString category);
                 void toggleFloatingValues();
                 // TODO: Expose this to user or not?
                 // TODO: TO PIMPL --->
                 void structureChanged();
                 void valueChanged();
                 void hoverEnter(QPoint pos);
                 void hoverLeave();
                 void showToolTip(QPoint pos, QString tip);  // Private signal
                 // <--- TO PIMPL
             public Q_SLOTS:
                 // These are for internal communication
                 // TODO: TO PIMPL --->
                 void barHoverEnterEvent(QPoint pos);
                 void barHoverLeaveEvent();
                 // <--- TO PIMPL
             private:
                 QString mName;
                 QList<qreal> mValues;   // TODO: replace with map (category, value)
-                QMap<QString,qreal> mMappedValues;
+                QMap<QString, qreal> mMappedValues;
                 QPen mPen;
                 QBrush mBrush;
                 QPen mFloatingValuePen;
             };
             QTCOMMERCIALCHART_END_NAMESPACE
             #endif // QBARSET_H

src/barchart/qpercentbarseries.h +1 -3

             #ifndef PERCENTBARSERIES_H
             #define PERCENTBARSERIES_H
             #include <QStringList>
             #include <qbarseries.h>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             class QTCOMMERCIALCHART_EXPORT QPercentBarSeries : public QBarSeries
             {
                 Q_OBJECT
             public:
-                QPercentBarSeries(QStringList categories, QObject* parent=0);
+                QPercentBarSeries(QStringList categories, QObject *parent = 0);
                 virtual QSeriesType type() const { return QSeries::SeriesTypePercentBar; }
             };
             QTCOMMERCIALCHART_END_NAMESPACE
             #endif // PERCENTBARSERIES_H

src/barchart/qstackedbarseries.h +1 -2

             #ifndef STACKEDBARSERIES_H
             #define STACKEDBARSERIES_H
             #include <QStringList>
             #include <qbarseries.h>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             class QTCOMMERCIALCHART_EXPORT QStackedBarSeries : public QBarSeries
             {
                 Q_OBJECT
             public:
-                QStackedBarSeries(QStringList categories, QObject* parent=0);
+                QStackedBarSeries(QStringList categories, QObject *parent = 0);
                 virtual QSeriesType type() const { return QSeries::SeriesTypeStackedBar; }
             };
             QTCOMMERCIALCHART_END_NAMESPACE
             #endif // STACKEDBARSERIES_H

src/barchart/stackedbarchartitem.cpp +9 -10

             #include "stackedbarchartitem_p.h"
             #include "bar_p.h"
             #include "barvalue_p.h"
             #include "qbarset.h"
             #include <QDebug>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             StackedBarChartItem::StackedBarChartItem(QBarSeries *series, ChartPresenter *presenter) :
-                BarChartItem(series,presenter)
+                BarChartItem(series, presenter)
             {
             }
             StackedBarChartItem::~StackedBarChartItem()
             {
             }
             QVector<QRectF> StackedBarChartItem::calculateLayout()
             {
                 QVector<QRectF> layout;
                 // Use temporary qreals for accurancy (we might get some compiler warnings... :)
                 qreal maxSum = mSeries->maxCategorySum();
                 // Domain:
                 if (mDomainMaxY > maxSum) {
                     maxSum = mDomainMaxY;
                 }
                 qreal height = geometry().height();
                 qreal width = geometry().width();
                 qreal scale = (height /  mSeries->maxCategorySum());
                 qreal categotyCount = mSeries->categoryCount();
-                qreal barWidth = width / (categotyCount *2);
+                qreal barWidth = width / (categotyCount * 2);
-                qreal xStep = width/categotyCount;
+                qreal xStep = width / categotyCount;
-                qreal xPos = xStep/2 - barWidth/2;
+                qreal xPos = xStep / 2 - barWidth / 2;
                 int itemIndex(0);
                 for (int category = 0; category < categotyCount; category++) {
                     qreal yPos = height;
                     for (int set=0; set < mSeries->barsetCount(); set++) {
                         qreal barHeight = mSeries->valueAt(set, category) * scale;
                         Bar* bar = mBars.at(itemIndex);
                         bar->setPen(mSeries->barsetAt(set)->pen());
                         bar->setBrush(mSeries->barsetAt(set)->brush());
-                        QRectF rect(xPos,yPos-barHeight,barWidth,barHeight);
+                        QRectF rect(xPos, yPos-barHeight, barWidth, barHeight);
                         layout.append(rect);
                         itemIndex++;
                         yPos -= barHeight;
                     }
                     xPos += xStep;
                 }
                 // Position floating values
                 itemIndex = 0;
                 xPos = (width/categotyCount);
                 for (int category=0; category < mSeries->categoryCount(); category++) {
                     qreal yPos = height;
                     for (int set=0; set < mSeries->barsetCount(); set++) {
-                        qreal barHeight = mSeries->valueAt(set,category) * scale;
+                        qreal barHeight = mSeries->valueAt(set, category) * scale;
                         BarValue* value = mFloatingValues.at(itemIndex);
                         QBarSet* barSet = mSeries->barsetAt(set);
-                        value->resize(100,50);  // TODO: proper layout for this.
+                        value->resize(100, 50);  // TODO: proper layout for this.
-                        value->setPos(xPos, yPos-barHeight/2);
+                        value->setPos(xPos, yPos-barHeight / 2);
                         value->setPen(barSet->floatingValuePen());
-                        if (mSeries->valueAt(set,category) != 0) {
+                        if (mSeries->valueAt(set, category) != 0) {
                             value->setValueString(QString::number(mSeries->valueAt(set,category)));
                         } else {
                             value->setValueString(QString(""));
                         }
                         itemIndex++;
                         yPos -= barHeight;
                     }
                     xPos += xStep;
                 }
                 return layout;
             }
             #include "moc_stackedbarchartitem_p.cpp"
             QTCOMMERCIALCHART_END_NAMESPACE

src/qlegend.cpp +69 -65

             #include "qchartglobal.h"
             #include "qlegend.h"
             #include "qseries.h"
             #include "legendmarker_p.h"
             #include "legendscrollbutton_p.h"
             #include "qxyseries.h"
             #include "qlineseries.h"
             #include "qareaseries.h"
             #include "qscatterseries.h"
             #include "qsplineseries.h"
             #include "qbarseries.h"
             #include "qstackedbarseries.h"
             #include "qpercentbarseries.h"
             #include "qbarset.h"
             #include "qpieseries.h"
             #include "qpieslice.h"
             #include "chartpresenter_p.h"
             #include <QPainter>
             #include <QPen>
             #include <QGraphicsSceneEvent>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             /*!
                 \class QLegend
                 \brief part of QtCommercial chart API.
                 QLegend is a graphical object, whics displays legend of the chart. Legend state is updated by QChart, when
                 series have been changed. By default, legend is drawn by QChart, but user can set a new parent to legend and
                 handle the drawing manually.
                 User isn't supposed to create or delete legend objects, but can reference it via QChart class.
                 \mainclass
                 \sa QChart, QSeries
             */
             /*!
                 \enum QLegend::PreferredLayout
                 This enum describes the possible position for legend inside chart.
                 \value PreferredLayoutTop
                 \value PreferredLayoutBottom
                 \value PreferredLayoutLeft
                 \value PreferredLayoutRight
             */
             /*!
                 \fn void QLegend::clicked(QSeries* series, Qt::MouseButton button)
                 \brief Notifies when series has been clicked on legend \a series \a button
             */
             /*!
                 \fn void QLegend::clicked(QBarSet* barset, Qt::MouseButton button)
                 \brief Notifies when barset has been clicked on legend \a barset \a button
             */
             /*!
                 \fn void QLegend::clicked(QPieSlice* slice, Qt::MouseButton button)
                 \brief Notifies when pie slice has been clicked on legend \a slice \a button
             */
             /*!
                 Constructs the legend object and sets the parent to \a parent
             */
             QLegend::QLegend(QGraphicsItem *parent)
                 : QGraphicsObject(parent)
                 ,mPos(0,0)
                 ,mSize(0,0)
                 ,mMinimumSize(50,20)                // TODO: magic numbers
                 ,mMaximumSize(150,100)
                 ,m_brush(Qt::darkGray)       // TODO: from theme?
                 ,mPreferredLayout(QLegend::PreferredLayoutTop)
                 ,mFirstMarker(0)
                 ,mMargin(5)
             {
             //    setVisible(false);
                 mScrollButtonLeft = new LegendScrollButton(LegendScrollButton::ScrollButtonIdLeft, this);
                 mScrollButtonRight = new LegendScrollButton(LegendScrollButton::ScrollButtonIdRight, this);
                 mScrollButtonUp = new LegendScrollButton(LegendScrollButton::ScrollButtonIdUp, this);
                 mScrollButtonDown = new LegendScrollButton(LegendScrollButton::ScrollButtonIdDown, this);
-                connect(mScrollButtonLeft,SIGNAL(clicked(QGraphicsSceneMouseEvent*)),this,SLOT(handleScrollButtonClicked(QGraphicsSceneMouseEvent*)));
+                connect(mScrollButtonLeft, SIGNAL(clicked(QGraphicsSceneMouseEvent*)),
-                connect(mScrollButtonRight,SIGNAL(clicked(QGraphicsSceneMouseEvent*)),this,SLOT(handleScrollButtonClicked(QGraphicsSceneMouseEvent*)));
+                        this, SLOT(handleScrollButtonClicked(QGraphicsSceneMouseEvent*)));
-                connect(mScrollButtonUp,SIGNAL(clicked(QGraphicsSceneMouseEvent*)),this,SLOT(handleScrollButtonClicked(QGraphicsSceneMouseEvent*)));
+                connect(mScrollButtonRight, SIGNAL(clicked(QGraphicsSceneMouseEvent*)),
-                connect(mScrollButtonDown,SIGNAL(clicked(QGraphicsSceneMouseEvent*)),this,SLOT(handleScrollButtonClicked(QGraphicsSceneMouseEvent*)));
+                        this, SLOT(handleScrollButtonClicked(QGraphicsSceneMouseEvent*)));
+                connect(mScrollButtonUp, SIGNAL(clicked(QGraphicsSceneMouseEvent*)),
+                        this, SLOT(handleScrollButtonClicked(QGraphicsSceneMouseEvent*)));
+                connect(mScrollButtonDown, SIGNAL(clicked(QGraphicsSceneMouseEvent*)),
+                        this, SLOT(handleScrollButtonClicked(QGraphicsSceneMouseEvent*)));
                 setZValue(ChartPresenter::LegendZValue);
             }
             /*!
                 Paints the legend to given \a painter. Paremeters \a option and \a widget arent used.
             */
             void QLegend::paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget)
             {
                 Q_UNUSED(option)
                 Q_UNUSED(widget)
                 painter->setOpacity(opacity());
                 painter->setPen(m_pen);
                 painter->setBrush(m_brush);
                 painter->drawRect(boundingRect());
             }
             /*!
                 Bounding rect of legend.
             */
             QRectF QLegend::boundingRect() const
             {
                 return QRectF(mPos,mSize);
             }
             /*!
                 Sets the \a brush of legend. Brush affects the background of legend.
             */
-            void QLegend::setBrush(const QBrush& brush)
+            void QLegend::setBrush(const QBrush &brush)
             {
-                if(m_brush!=brush){
+                if (m_brush != brush) {
                     m_brush = brush;
                     update();
                 }
             }
             /*!
                 Returns the brush used by legend.
             */
             QBrush QLegend::brush() const
             {
                 return m_brush;
             }
             /*!
                 Sets the \a pen of legend. Pen affects the legend borders.
             */
-            void QLegend::setPen(const QPen& pen)
+            void QLegend::setPen(const QPen &pen)
             {
-                if(m_pen!=pen){
+                if (m_pen != pen) {
                     m_pen = pen;
                     update();
                 }
             }
             /*!
                 Returns the pen used by legend
             */
             QPen QLegend::pen() const
             {
                 return m_pen;
             }
             /*!
                 Sets the \a preferred layout for legend. Legend tries to paint itself on the defined position in chart.
                 \sa QLegend::PreferredLayout
             */
             void QLegend::setPreferredLayout(QLegend::PreferredLayout preferred)
             {
                 mPreferredLayout = preferred;
                 updateLayout();
             }
             /*!
                 Returns the preferred layout for legend
             */
             QLegend::PreferredLayout QLegend::preferredLayout() const
             {
                 return mPreferredLayout;
             }
             /*!
                 Returns the maximum size of legend.
             */
             QSizeF QLegend::maximumSize() const
             {
                 return mMaximumSize;
             }
             /*!
                 Sets the maximum \a size for legend. The legend can't grow bigger than this size. If there are
                 more series than legend can fit to this size, scroll buttons are displayed.
             */
             void QLegend::setMaximumSize(const QSizeF size)
             {
                 mMaximumSize = size;
                 updateLayout();
             }
             /*!
                 Returns the current size of legend.
             */
             QSizeF QLegend::size() const
             {
                 return mSize;
             }
             /*!
                 Sets the \a size of legend. If size is bigger than maximum size of legend, the legend is resized to the maximum size.
                 \sa setMmaximumSize()
             */
             void QLegend::setSize(const QSizeF size)
             {
                 mSize = size;
                 if (mSize.width() > mMaximumSize.width()) {
                     mSize.setWidth(mMaximumSize.width());
                 }
                 if (mSize.height() > mMaximumSize.height()) {
                     mSize.setHeight(mMaximumSize.height());
                 }
             }
             /*!
                 Sets position of legend to \a pos
             */
             void QLegend::setPos(const QPointF &pos)
             {
                 mPos = pos;
                 updateLayout();
             }
             /*!
                 \internal \a series \a domain Should be called when series is added to chart.
             */
-            void QLegend::handleSeriesAdded(QSeries* series, Domain* domain)
+            void QLegend::handleSeriesAdded(QSeries *series, Domain *domain)
             {
                 Q_UNUSED(domain)
                 createMarkers(series);
                 connectSeries(series);
                 updateLayout();
             }
             /*!
                 \internal \a series Should be called when series is removed from chart.
             */
-            void QLegend::handleSeriesRemoved(QSeries* series)
+            void QLegend::handleSeriesRemoved(QSeries *series)
             {
                 disconnectSeries(series);
                 if (series->type() == QSeries::SeriesTypeArea)
                 {
                     // This is special case. Area series has upper and lower series, which each have markers
-                    QAreaSeries* s = static_cast<QAreaSeries*> (series);
+                    QAreaSeries* s = static_cast<QAreaSeries *> (series);
                     deleteMarkers(s->upperSeries());
                     deleteMarkers(s->lowerSeries());
                 } else {
                     deleteMarkers(series);
                 }
                 updateLayout();
             }
             /*!
                 \internal \a slices Should be called when slices are added to pie chart.
             */
-            void QLegend::handleAdded(QList<QPieSlice*> slices)
+            void QLegend::handleAdded(QList<QPieSlice *> slices)
             {
-                QPieSeries* series = static_cast<QPieSeries*> (sender());
+                QPieSeries* series = static_cast<QPieSeries *> (sender());
                 foreach(QPieSlice* s, slices) {
-                    LegendMarker* marker = new LegendMarker(series,s,this);
+                    LegendMarker* marker = new LegendMarker(series, s, this);
                     marker->setName(s->label());
                     marker->setBrush(s->sliceBrush());
-                    connect(marker,SIGNAL(clicked(QPieSlice*,Qt::MouseButton)),this,SIGNAL(clicked(QPieSlice*,Qt::MouseButton)));
+                    connect(marker, SIGNAL(clicked(QPieSlice*,Qt::MouseButton)),
-                    connect(s,SIGNAL(changed()),marker,SLOT(changed()));
+                            this, SIGNAL(clicked(QPieSlice*,Qt::MouseButton)));
-                    connect(s,SIGNAL(destroyed()),marker,SLOT(deleteLater()));
+                    connect(s, SIGNAL(changed()), marker, SLOT(changed()));
-                    connect(marker,SIGNAL(destroyed()),this,SLOT(handleMarkerDestroyed()));
+                    connect(s, SIGNAL(destroyed()), marker, SLOT(deleteLater()));
+                    connect(marker, SIGNAL(destroyed()), this, SLOT(handleMarkerDestroyed()));
                     mMarkers.append(marker);
                     childItems().append(marker);
                 }
                 updateLayout();
             }
             /*!
                 \internal \a slices Should be called when slices are removed from pie chart. Currently unused,
                 because removed slices are also deleted and we listen destroyed signal
             */
             void QLegend::handleRemoved(QList<QPieSlice *> slices)
             {
                 Q_UNUSED(slices)
                 // Propably no need to listen for this, since removed slices are deleted and we listen destroyed signal
             //    qDebug() << "QLegend::handleRemoved(QList<QPieSlice*> slices) count:" << slices.count();
             }
             /*!
                 \internal Notifies legend that some marker has been removed. Sent by legend markers when destroyed
             */
             void QLegend::handleMarkerDestroyed()
             {
                 // TODO: what if more than one markers are destroyed and we update layout after first one?
-                LegendMarker* m = static_cast<LegendMarker*> (sender());
+                LegendMarker* m = static_cast<LegendMarker *> (sender());
                 mMarkers.removeOne(m);
                 updateLayout();
             }
             /*!
                 \internal \a event Handles clicked signals from scroll buttons
             */
             void QLegend::handleScrollButtonClicked(QGraphicsSceneMouseEvent *event)
             {
                 Q_UNUSED(event);    // Maybe later something happens with right click...
-                LegendScrollButton* scrollButton = static_cast<LegendScrollButton*> (sender());
+                LegendScrollButton* scrollButton = static_cast<LegendScrollButton *> (sender());
                 Q_ASSERT(scrollButton);
                 switch (scrollButton->id()) {
                 case LegendScrollButton::ScrollButtonIdLeft:
                 case LegendScrollButton::ScrollButtonIdUp: {
                     // Lower limit is same in these cases
                     mFirstMarker--;
                     checkFirstMarkerBounds();
                     break;
                 }
                 case LegendScrollButton::ScrollButtonIdRight:
                 case LegendScrollButton::ScrollButtonIdDown: {
                     mFirstMarker++;
                     checkFirstMarkerBounds();
                     break;
                 }
                 default: {
                     break;
                 }
                 }
                 updateLayout();
             }
             /*!
                 \internal Connects the \a series to legend. Legend listens changes in series, for example pie slices added / removed.
                 Not all series notify about events
             */
             void QLegend::connectSeries(QSeries *series)
             {
                 // Connect relevant signals from series
                 switch (series->type())
                 {
                 case QSeries::SeriesTypeLine: {
             //        QLineSeries* lineSeries = static_cast<QLineSeries*>(series);
                     break;
                 }
                 case QSeries::SeriesTypeArea: {
             //        QAreaSeries* areaSeries = static_cast<QAreaSeries*>(series);
                     break;
                 }
                 case QSeries::SeriesTypeBar: {
             //        QBarSeries* barSeries = static_cast<QBarSeries*>(series);
                     break;
                 }
                 case QSeries::SeriesTypeStackedBar: {
             //        QStackedBarSeries* stackedBarSeries = static_cast<QStackedBarSeries*>(series);
                     break;
                 }
                 case QSeries::SeriesTypePercentBar: {
             //        QPercentBarSeries* percentBarSeries = static_cast<QPercentBarSeries*>(series);
                     break;
                 }
                 case QSeries::SeriesTypeScatter: {
             //        QScatterSeries *scatterSeries = static_cast<QScatterSeries *>(series);
                     break;
                 }
                 case QSeries::SeriesTypePie: {
                     QPieSeries *pieSeries = static_cast<QPieSeries *>(series);
                     connect(pieSeries,SIGNAL(added(QList<QPieSlice*>)),this,SLOT(handleAdded(QList<QPieSlice*>)));
             //        connect(pieSeries,SIGNAL(removed(QList<QPieSlice*>)),this,SLOT(handleRemoved(QList<QPieSlice*>)));
                     break;
                 }
                 case QSeries::SeriesTypeSpline: {
             //        QSplineSeries* splineSeries = static_cast<QSplineSeries*>(series);
                     break;
                 }
                 default: {
                     qDebug()<< "QLegend::connectSeries" << series->type() << "not implemented.";
                     break;
                 }
                 }
             }
             /*!
                 \internal Disconnects \a series from legend. No more status updates from series to legend.
             */
             void QLegend::disconnectSeries(QSeries *series)
             {
                 // Connect relevant signals from series
                 switch (series->type())
                 {
                 case QSeries::SeriesTypeLine: {
             //        QLineSeries* lineSeries = static_cast<QLineSeries*>(series);
                     break;
                 }
                 case QSeries::SeriesTypeArea: {
             //        QAreaSeries* areaSeries = static_cast<QAreaSeries*>(series);
                     break;
                 }
                 case QSeries::SeriesTypeBar: {
             //        QBarSeries* barSeries = static_cast<QBarSeries*>(series);
                     break;
                 }
                 case QSeries::SeriesTypeStackedBar: {
             //        QStackedBarSeries* stackedBarSeries = static_cast<QStackedBarSeries*>(series);
                     break;
                 }
                 case QSeries::SeriesTypePercentBar: {
             //        QPercentBarSeries* percentBarSeries = static_cast<QPercentBarSeries*>(series);
                     break;
                 }
                 case QSeries::SeriesTypeScatter: {
             //        QScatterSeries *scatterSeries = static_cast<QScatterSeries *>(series);
                     break;
                 }
                 case QSeries::SeriesTypePie: {
                     QPieSeries *pieSeries = static_cast<QPieSeries *>(series);
-                    disconnect(pieSeries,SIGNAL(added(QList<QPieSlice*>)),this,SLOT(handleAdded(QList<QPieSlice*>)));
+                    disconnect(pieSeries, SIGNAL(added(QList<QPieSlice *>)), this, SLOT(handleAdded(QList<QPieSlice *>)));
             //        disconnect(pieSeries,SIGNAL(removed(QList<QPieSlice*>)),this,SLOT(handleRemoved(QList<QPieSlice*>)));
                     break;
                 }
                 case QSeries::SeriesTypeSpline: {
             //        QSplineSeries* splineSeries = static_cast<QSplineSeries*>(series);
                     break;
                 }
                 default: {
                     qDebug()<< "QLegend::disconnectSeries" << series->type() << "not implemented.";
                     break;
                 }
                 }
             }
             /*!
                 \internal Creates new markers for \a series. Marker contains the colored rectangle and series name.
                 With pie chart, created markers depend on pie slices.
                 With bar chart, created markers depend on bar sets.
             */
             void QLegend::createMarkers(QSeries *series)
             {
                 switch (series->type())
                 {
                 case QSeries::SeriesTypeLine: {
-                    QLineSeries* lineSeries = static_cast<QLineSeries*>(series);
+                    QLineSeries *lineSeries = static_cast<QLineSeries *>(series);
                     appendMarkers(lineSeries);
                     break;
                 }
                 case QSeries::SeriesTypeArea: {
-                    QAreaSeries* areaSeries = static_cast<QAreaSeries*>(series);
+                    QAreaSeries *areaSeries = static_cast<QAreaSeries *>(series);
                     appendMarkers(areaSeries->upperSeries());
                     if(areaSeries->lowerSeries())
-                    appendMarkers(areaSeries->lowerSeries());
+                        appendMarkers(areaSeries->lowerSeries());
                     break;
                 }
                 case QSeries::SeriesTypeBar: {
-                    QBarSeries* barSeries = static_cast<QBarSeries*>(series);
+                    QBarSeries *barSeries = static_cast<QBarSeries *>(series);
                     appendMarkers(barSeries);
                     break;
                 }
                 case QSeries::SeriesTypeStackedBar: {
-                    QStackedBarSeries* stackedBarSeries = static_cast<QStackedBarSeries*>(series);
+                    QStackedBarSeries *stackedBarSeries = static_cast<QStackedBarSeries *>(series);
                     appendMarkers(stackedBarSeries);
                     break;
                 }
                 case QSeries::SeriesTypePercentBar: {
-                    QPercentBarSeries* percentBarSeries = static_cast<QPercentBarSeries*>(series);
+                    QPercentBarSeries *percentBarSeries = static_cast<QPercentBarSeries *>(series);
                     appendMarkers(percentBarSeries);
                     break;
                 }
                 case QSeries::SeriesTypeScatter: {
                     QScatterSeries *scatterSeries = static_cast<QScatterSeries *>(series);
                     appendMarkers(scatterSeries);
                     break;
                 }
                 case QSeries::SeriesTypePie: {
                     QPieSeries *pieSeries = static_cast<QPieSeries *>(series);
                     appendMarkers(pieSeries);
                     break;
                 }
                 case QSeries::SeriesTypeSpline: {
-                    QSplineSeries* splineSeries = static_cast<QSplineSeries*>(series);
+                    QSplineSeries *splineSeries = static_cast<QSplineSeries *>(series);
                     appendMarkers(splineSeries);
                     break;
                 }
                 default: {
                     qDebug()<< "QLegend::createMarkers" << series->type() << "not implemented.";
                     break;
                 }
                 }
             }
             /*!
                 \internal Helper function. Appends markers from \a series to legend.
             */
             void QLegend::appendMarkers(QXYSeries* series)
             {
                 LegendMarker* marker = new LegendMarker(series,this);
                 marker->setName(series->name());
                 marker->setPen(series->pen());
                 marker->setBrush(series->brush());
-                connect(marker,SIGNAL(clicked(QSeries*,Qt::MouseButton)),this,SIGNAL(clicked(QSeries*,Qt::MouseButton)));
+                connect(marker, SIGNAL(clicked(QSeries *, Qt::MouseButton)), this, SIGNAL(clicked(QSeries *, Qt::MouseButton)));
-                connect(marker,SIGNAL(destroyed()),this,SLOT(handleMarkerDestroyed()));
+                connect(marker, SIGNAL(destroyed()), this, SLOT(handleMarkerDestroyed()));
                 mMarkers.append(marker);
                 childItems().append(marker);
             }
             /*!
                 \internal Helper function. Appends markers from  \a series to legend.
             */
             void QLegend::appendMarkers(QBarSeries *series)
             {
                 foreach(QBarSet* s, series->barSets()) {
-                    LegendMarker* marker = new LegendMarker(series,s,this);
+                    LegendMarker* marker = new LegendMarker(series, s, this);
                     marker->setName(s->name());
                     marker->setPen(s->pen());
                     marker->setBrush(s->brush());
-                    connect(marker,SIGNAL(clicked(QBarSet*,Qt::MouseButton)),this,SIGNAL(clicked(QBarSet*,Qt::MouseButton)));
+                    connect(marker, SIGNAL(clicked(QBarSet *, Qt::MouseButton)),
-                    connect(s,SIGNAL(valueChanged()),marker,SLOT(changed()));
+                            this, SIGNAL(clicked(QBarSet *, Qt::MouseButton)));
-                    connect(marker,SIGNAL(destroyed()),this,SLOT(handleMarkerDestroyed()));
+                    connect(s, SIGNAL(valueChanged()), marker, SLOT(changed()));
+                    connect(marker, SIGNAL(destroyed()), this, SLOT(handleMarkerDestroyed()));
                     mMarkers.append(marker);
                     childItems().append(marker);
                 }
             }
             /*!
                 \internal Helper function. Appends markers from  \a series to legend.
             */
             void QLegend::appendMarkers(QPieSeries *series)
             {
                 foreach(QPieSlice* s, series->slices()) {
-                    LegendMarker* marker = new LegendMarker(series,s,this);
+                    LegendMarker* marker = new LegendMarker(series, s, this);
                     marker->setName(s->label());
                     marker->setPen(s->slicePen());
                     marker->setBrush(s->sliceBrush());
-                    connect(marker,SIGNAL(clicked(QPieSlice*,Qt::MouseButton)),this,SIGNAL(clicked(QPieSlice*,Qt::MouseButton)));
+                    connect(marker, SIGNAL(clicked(QPieSlice *, Qt::MouseButton)),
-                    connect(s,SIGNAL(changed()),marker,SLOT(changed()));
+                            this, SIGNAL(clicked(QPieSlice *, Qt::MouseButton)));
-                    connect(s,SIGNAL(destroyed()),marker,SLOT(deleteLater()));
+                    connect(s, SIGNAL(changed()), marker, SLOT(changed()));
-                    connect(marker,SIGNAL(destroyed()),this,SLOT(handleMarkerDestroyed()));
+                    connect(s, SIGNAL(destroyed()), marker, SLOT(deleteLater()));
+                    connect(marker, SIGNAL(destroyed()), this, SLOT(handleMarkerDestroyed()));
                     mMarkers.append(marker);
                     childItems().append(marker);
                 }
             }
             /*!
                 \internal Deletes all markers that are created from \a series
             */
             void QLegend::deleteMarkers(QSeries *series)
             {
                 // Search all markers that belong to given series and delete them.
                 foreach (LegendMarker *m, mMarkers) {
                     if (m->series() == series) {
                         mMarkers.removeOne(m);
                         delete m;
                     }
                 }
             }
             /*!
                 \internal Updates layout of legend. Tries to fit as many markers as possible up to the maximum size of legend.
                 If items don't fit, sets the visibility of scroll buttons accordingly.
                 Causes legend to be resized.
             */
             void QLegend::updateLayout()
             {
                 // Calculate layout for markers and text
                 if (mMarkers.count() <= 0) {
                     // Nothing to do
                     return;
                 }
                 // Find out widest item.
                 QSizeF markerMaxSize = maximumMarkerSize();
                 checkFirstMarkerBounds();
                 // Use max height as scroll button size
                 rescaleScrollButtons(QSize(markerMaxSize.height() ,markerMaxSize.height()));
                 qreal totalWidth = 0;
                 qreal totalHeight = 0;
                 switch (mPreferredLayout)
                 {
                 // Both cases organise items horizontally
                 case QLegend::PreferredLayoutBottom:
                 case QLegend::PreferredLayoutTop: {
                     qreal xStep = markerMaxSize.width();
                     qreal x = mPos.x() + mMargin;
                     qreal y = mPos.y() + mMargin;
                     int column = 0;
                     int maxColumns = 1;
                     qreal scrollButtonWidth = 0;
                     // Set correct visibility for scroll scrollbuttons
                     if (scrollButtonsVisible()) {
                         mScrollButtonLeft->setVisible(true);
                         mScrollButtonRight->setVisible(true);
-                        totalWidth += (mScrollButtonLeft->boundingRect().width() + mMargin) * 2;   // scrollbuttons visible, so add their width to total width
+                        // scrollbuttons visible, so add their width to total width
+                        totalWidth += (mScrollButtonLeft->boundingRect().width() + mMargin) * 2;
                         scrollButtonWidth = mScrollButtonLeft->boundingRect().width() + mMargin;
-                        x += scrollButtonWidth;                  // start position changes by scrollbutton width
+                        // start position changes by scrollbutton width
+                        x += scrollButtonWidth;
                     } else {
                         mScrollButtonLeft->setVisible(false);
                         mScrollButtonRight->setVisible(false);
                     }
                     mScrollButtonUp->setVisible(false);
                     mScrollButtonDown->setVisible(false);
-                    for (int i=0; i<mMarkers.count(); i++) {
+                    for (int i=0; i < mMarkers.count(); i++) {
-                        LegendMarker* m = mMarkers.at(i);
+                        LegendMarker *m = mMarkers.at(i);
-                        if (i<mFirstMarker) {
+                        if (i < mFirstMarker) {
                             // Markers before first are not visible.
                             m->setVisible(false);
                         } else {
                             if ((x + xStep + scrollButtonWidth + mMargin) > (mPos.x() + mMaximumSize.width())) {
                                 // This marker would go outside legend rect.
                                 m->setVisible(false);
                             } else {
                                 // This marker is ok
                                 m->setVisible(true);
-                                m->setPos(x,y);
+                                m->setPos(x, y);
                                 x += xStep;
                                 column++;
                             }
                         }
                         maxColumns = column;
                     }
                     mScrollButtonLeft->setPos(mPos.x() + mMargin, y);
-                    mScrollButtonRight->setPos(x+mMargin,y);
+                    mScrollButtonRight->setPos(x + mMargin, y);
                     totalWidth += maxColumns * markerMaxSize.width() + mMargin * 2;
                     totalHeight = markerMaxSize.height() + mMargin * 2;
                     break;
                 }
                 // Both cases organize items vertically
                 case QLegend::PreferredLayoutLeft:
                 case QLegend::PreferredLayoutRight: {
                     qreal yStep = markerMaxSize.height();
                     qreal x = mPos.x() + mMargin;
                     qreal y = mPos.y() + mMargin;
                     int row = 1;
                     int maxRows = 1;
                     qreal scrollButtonHeight = 0;
                     // Set correct visibility for scroll scrollbuttons
                     if (scrollButtonsVisible()) {
                         mScrollButtonUp->setVisible(true);
                         mScrollButtonDown->setVisible(true);
                         totalHeight += (mScrollButtonUp->boundingRect().height() + mMargin) * 2;   // scrollbuttons visible, so add their height to total height
                         scrollButtonHeight = mScrollButtonUp->boundingRect().height();
                         y += scrollButtonHeight + mMargin;                                         // start position changes by scrollbutton height
                     } else {
                         mScrollButtonUp->setVisible(false);
                         mScrollButtonDown->setVisible(false);
                     }
                     mScrollButtonLeft->setVisible(false);
                     mScrollButtonRight->setVisible(false);
-                    for (int i=0; i<mMarkers.count(); i++) {
+                    for (int i=0; i < mMarkers.count(); i++) {
                         LegendMarker* m = mMarkers.at(i);
-                        if (i<mFirstMarker) {
+                        if (i < mFirstMarker) {
                             // Markers before first are not visible.
                             m->setVisible(false);
                         } else {
                             if ((y + yStep + scrollButtonHeight) > (mPos.y() + mMaximumSize.height())) {
                                 // This marker would go outside legend rect.
                                 m->setVisible(false);
                             } else {
                                 // This marker is ok
                                 m->setVisible(true);
-                                m->setPos(x,y);
+                                m->setPos(x, y);
                                 y += yStep;
                                 row++;
                             }
                         }
                         maxRows = row;
                     }
                     mScrollButtonUp->setPos(mPos.x() + mMargin, mPos.y() + mMargin);
                     mScrollButtonDown->setPos(mPos.x() + mMargin, y + mMargin);
                     totalWidth += markerMaxSize.width() + mMargin * 2;
                     totalHeight = maxRows * markerMaxSize.height() + mMargin * 4 + scrollButtonHeight; // TODO: check this
                     break;
                 }
                 default: {
                     break;
                 }
                 }
                 mSize.setWidth(totalWidth);
                 mSize.setHeight(totalHeight);
                 update();
             }
             /*!
                 \internal Sets the size of scroll buttons to \a size
             */
             void QLegend::rescaleScrollButtons(const QSize &size)
             {
                 QPolygonF left;
-                left << QPointF(size.width(),0) << QPointF(0,size.height()/2) << QPointF(size.width(),size.height());
+                left << QPointF(size.width(), 0) << QPointF(0, size.height() / 2) << QPointF(size.width(), size.height());
                 QPolygonF right;
-                right << QPointF(0,0) << QPointF(size.width(),size.height()/2) << QPointF(0,size.height());
+                right << QPointF(0, 0) << QPointF(size.width(), size.height() / 2) << QPointF(0, size.height());
                 QPolygonF up;
-                up << QPointF(0,size.height()) << QPointF(size.width()/2,0) << QPointF(size.width(),size.height());
+                up << QPointF(0, size.height()) << QPointF(size.width() / 2,0) << QPointF(size.width(), size.height());
                 QPolygonF down;
-                down << QPointF(0,0) << QPointF(size.width()/2,size.height()) << QPointF(size.width(),0);
+                down << QPointF(0, 0) << QPointF(size.width() / 2, size.height()) << QPointF(size.width(), 0);
                 mScrollButtonLeft->setPolygon(left);
                 mScrollButtonRight->setPolygon(right);
                 mScrollButtonUp->setPolygon(up);
                 mScrollButtonDown->setPolygon(down);
             }
             /*!
                 \internal Finds out maximum size of single marker. Marker sizes depend on series names.
             */
             QSizeF QLegend::maximumMarkerSize()
             {
                 QSizeF max(0,0);
                 foreach (LegendMarker* m, mMarkers) {
-                    if (m->boundingRect().width() > max.width()) {
+                    if (m->boundingRect().width() > max.width())
                         max.setWidth(m->boundingRect().width());
+                    if (m->boundingRect().height() > max.height())
-                    if (m->boundingRect().height() > max.height()) {
                         max.setHeight(m->boundingRect().height());
                 }
                 return max;
             }
             /*!
                 \internal Checks that first marker is in acceptable bounds. Bounds range from 0 to (maximum number of markers - visible markers)
                 If scrollbuttons are visible, they affect the number of visible markers.
             */
             void QLegend::checkFirstMarkerBounds()
             {
                 if ((mPreferredLayout == QLegend::PreferredLayoutLeft) || (mPreferredLayout == QLegend::PreferredLayoutRight)) {
                     // Bounds limited by height.
                     int max;
                     if (scrollButtonsVisible()) {
-                        max = (mMaximumSize.height() - mScrollButtonLeft->boundingRect().height() * 2 - mMargin*4) / maximumMarkerSize().height();
+                        max = (mMaximumSize.height() - mScrollButtonLeft->boundingRect().height() * 2 - mMargin * 4) / maximumMarkerSize().height();
                     } else {
                         max = mMaximumSize.height() / maximumMarkerSize().height();
                     }
-                    if (mFirstMarker > mMarkers.count() - max) {
+                    if (mFirstMarker > mMarkers.count() - max)
                         mFirstMarker = mMarkers.count() - max;
                 } else {
                     // Bounds limited by width
                     int max;
                     if (scrollButtonsVisible()) {
                         max = (mMaximumSize.width() - mScrollButtonLeft->boundingRect().width() * 2 - mMargin*4) / maximumMarkerSize().width();
                     } else {
                         max = mMaximumSize.width() / maximumMarkerSize().width();
                     }
-                    if (mFirstMarker > mMarkers.count() - max) {
+                    if (mFirstMarker > mMarkers.count() - max)
                         mFirstMarker = mMarkers.count() - max;
                 }
-                if (mFirstMarker < 0) {
+                if (mFirstMarker < 0)
                     mFirstMarker = 0;
             }
             /*!
                 \internal Helper function. Visibility of scroll buttons isn't quite obvious, so helper function clarifies the logic.
             */
             bool QLegend::scrollButtonsVisible()
             {
                 // Just a helper to clarify, what the magic below means :)
                 if ((mPreferredLayout == QLegend::PreferredLayoutTop) || (mPreferredLayout == QLegend::PreferredLayoutBottom)) {
                     return (maximumMarkerSize().width() * mMarkers.count() + mMargin * 2 > mMaximumSize.width());
                 } else if ((mPreferredLayout == QLegend::PreferredLayoutLeft) || (mPreferredLayout == QLegend::PreferredLayoutRight)) {
                     return (maximumMarkerSize().height() * mMarkers.count() + mMargin * 2 > mMaximumSize.height());
                 }
                 return (maximumMarkerSize().height() * mMarkers.count() + mMargin * 2 > mMaximumSize.height());
             }
             #include "moc_qlegend.cpp"
             QTCOMMERCIALCHART_END_NAMESPACE

src/qlegend.h +24 -24

             #ifndef QLEGEND_H
             #define QLEGEND_H
             #include <qchartglobal.h>
             #include <qseries.h>
             #include <QGraphicsObject>
             QTCOMMERCIALCHART_BEGIN_NAMESPACE
             class Domain;
             class LegendMarker;
             class QPieSlice;
             class QXYSeries;
             class QBarSet;
             class QBarSeries;
             class QPieSeries;
             class LegendScrollButton;
             class QTCOMMERCIALCHART_EXPORT QLegend : public QGraphicsObject
             {
                 Q_OBJECT
             public:
                 enum PreferredLayout {
                     PreferredLayoutTop,
                     PreferredLayoutBottom,
                     PreferredLayoutLeft,
-                    PreferredLayoutRight,
+                    PreferredLayoutRight
                 };
                 explicit QLegend(QGraphicsItem *parent = 0);
                 void paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget = 0);
                 QRectF boundingRect() const;
-                void setBrush(const QBrush& brush);
+                void setBrush(const QBrush &brush);
                 QBrush brush() const;
-                void setPen(const QPen& pen);
+                void setPen(const QPen &pen);
                 QPen pen() const;
                 void setPreferredLayout(QLegend::PreferredLayout preferred);
                 QLegend::PreferredLayout preferredLayout() const;
                 QSizeF maximumSize() const;
                 void setMaximumSize(const QSizeF size);
                 QSizeF size() const;
                 void setSize(const QSizeF size);
                 void setPos(const QPointF &pos);
             signals:
                 // for interactions.
-                void clicked(QSeries* series, Qt::MouseButton button);
+                void clicked(QSeries *series, Qt::MouseButton button);
-                void clicked(QBarSet* barset, Qt::MouseButton button);
+                void clicked(QBarSet *barset, Qt::MouseButton button);
-                void clicked(QPieSlice* slice, Qt::MouseButton button);
+                void clicked(QPieSlice *slice, Qt::MouseButton button);
             public slots:
                 // PIMPL --->
-                void handleSeriesAdded(QSeries* series,Domain* domain);
+                void handleSeriesAdded(QSeries *series, Domain *domain);
-                void handleSeriesRemoved(QSeries* series);
+                void handleSeriesRemoved(QSeries *series);
-                void handleAdded(QList<QPieSlice*> slices);
+                void handleAdded(QList<QPieSlice *> slices);
-                void handleRemoved(QList<QPieSlice*> slices);
+                void handleRemoved(QList<QPieSlice *> slices);
                 void handleMarkerDestroyed();
-                void handleScrollButtonClicked(QGraphicsSceneMouseEvent* event);
+                void handleScrollButtonClicked(QGraphicsSceneMouseEvent *event);
                 // PIMPL <---
             private:
                 // PIMPL --->
-                void connectSeries(QSeries* series);
+                void connectSeries(QSeries *series);
-                void disconnectSeries(QSeries* series);
+                void disconnectSeries(QSeries *series);
-                void createMarkers(QSeries* series);
+                void createMarkers(QSeries *series);
-                void appendMarkers(QXYSeries* series);      // All line series are derived from QXYSeries, so this works for now
+                void appendMarkers(QXYSeries *series);      // All line series are derived from QXYSeries, so this works for now
-                void appendMarkers(QBarSeries* series);
+                void appendMarkers(QBarSeries *series);
-                void appendMarkers(QPieSeries* series);
+                void appendMarkers(QPieSeries *series);
-                void deleteMarkers(QSeries* series);
+                void deleteMarkers(QSeries *series);
                 void updateLayout();
-                void rescaleScrollButtons(const QSize& size);
+                void rescaleScrollButtons(const QSize &size);
                 QSizeF maximumMarkerSize();
                 void checkFirstMarkerBounds();
                 bool scrollButtonsVisible();
                 QPointF mPos;
                 QSizeF mSize;
                 QSizeF mMinimumSize;
                 QSizeF mMaximumSize;
-                QList<LegendMarker*> mMarkers;
+                QList<LegendMarker *> mMarkers;
                 QBrush m_brush;
                 QPen m_pen;
                 QLegend::PreferredLayout mPreferredLayout;
                 int mFirstMarker;
-                LegendScrollButton* mScrollButtonLeft;
+                LegendScrollButton *mScrollButtonLeft;
-                LegendScrollButton* mScrollButtonRight;
+                LegendScrollButton *mScrollButtonRight;
-                LegendScrollButton* mScrollButtonUp;
+                LegendScrollButton *mScrollButtonUp;
-                LegendScrollButton* mScrollButtonDown;
+                LegendScrollButton *mScrollButtonDown;
                 qreal mMargin;
                 // <--- PIMPL
             };
             QTCOMMERCIALCHART_END_NAMESPACE
             #endif // QLEGEND_H

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages