mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2024-11-28 05:00:20 +00:00
Code Issues Projects Releases Packages Wiki Activity

Change the Star Widget to use QImage instead of QPixmap

Browse source 
Also, clean a few calls, this should make the widget a tiny
bit faster. This patch also moves the grayImage function from
the star widget to the global scope, so I can use it on the
Calendar widget.

Signed-off-by: Tomaz Canabrava <tomaz.canabrava@intel.com>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
This commit is contained in:
Tomaz Canabrava 2014-06-19 18:45:26 -03:00 committed by Dirk Hohndel
parent f3f03e2ee8
commit be462ae1a6
5 changed files with 30 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
qt-ui/modeldelegates.cpp
View file

@ -47,10 +47,13 @@ void StarWidgetsDelegate::paint(QPainter *painter, const QStyleOptionViewItem &o
int deltaY = option.rect.height() / 2 - StarWidget::starActive().height() / 2;
painter->save();
painter->setRenderHint(QPainter::Antialiasing, true);
const QPixmap active = QPixmap::fromImage(StarWidget::starActive());
const QPixmap inactive = QPixmap::fromImage(StarWidget::starInactive());
for (int i = 0; i < rating; i++)
painter->drawPixmap(option.rect.x() + i * IMG_SIZE + SPACING, option.rect.y() + deltaY, StarWidget::starActive());
painter->drawPixmap(option.rect.x() + i * IMG_SIZE + SPACING, option.rect.y() + deltaY, active);
for (int i = rating; i < TOTALSTARS; i++)
painter->drawPixmap(option.rect.x() + i * IMG_SIZE + SPACING, option.rect.y() + deltaY, StarWidget::starInactive());
painter->drawPixmap(option.rect.x() + i * IMG_SIZE + SPACING, option.rect.y() + deltaY, inactive);
painter->restore();
}

3
qt-ui/simplewidgets.cpp
View file

@ -332,7 +332,8 @@ void DateWidget::paintEvent(QPaintEvent *event)
static int pixRedTag = 18; /* calculated using a ruler. */
QPainter painter(this);
painter.drawPixmap(QPoint(0,0), pix);
painter.drawPixmap(QPoint(0,0), isEnabled() ? pix : QPixmap::fromImage(grayImage(pix.toImage())));
QString month = mDate.toString("MMM");
QString year = mDate.toString("yyyy");

1
qt-ui/simplewidgets.h
View file

@ -111,5 +111,6 @@ private:
};
bool isGnome3Session();
QImage grayImage(const QImage& coloredImg);
#endif // SIMPLEWIDGETS_H

35
qt-ui/starwidget.cpp
View file

@ -7,18 +7,19 @@
#include <unistd.h>
#include <QStyle>
#include <QStyleOption>
#include "simplewidgets.h"
QPixmap *StarWidget::activeStar = 0;
QPixmap *StarWidget::inactiveStar = 0;
QImage StarWidget::activeStar;
QImage StarWidget::inactiveStar;
QPixmap StarWidget::starActive()
const QImage& StarWidget::starActive()
{
return (*activeStar);
return activeStar;
}
QPixmap StarWidget::starInactive()
const QImage& StarWidget::starInactive()
{
return (*inactiveStar);
return inactiveStar;
}
int StarWidget::currentStars() const
@ -48,12 +49,14 @@ void StarWidget::mouseReleaseEvent(QMouseEvent *event)
void StarWidget::paintEvent(QPaintEvent *event)
{
QPainter p(this);
QPixmap active = QPixmap::fromImage(starActive());
QPixmap inactive = QPixmap::fromImage(starInactive());
for (int i = 0; i < current; i++)
p.drawPixmap(i * IMG_SIZE + SPACING, 0, starActive());
p.drawPixmap(i * IMG_SIZE + SPACING, 0, active);
for (int i = current; i < TOTALSTARS; i++)
p.drawPixmap(i * IMG_SIZE + SPACING, 0, starInactive());
p.drawPixmap(i * IMG_SIZE + SPACING, 0, inactive);
if (hasFocus()) {
QStyleOptionFocusRect option;
@ -74,8 +77,7 @@ StarWidget::StarWidget(QWidget *parent, Qt::WindowFlags f) : QWidget(parent, f),
current(0),
readOnly(false)
{
if (!activeStar) {
activeStar = new QPixmap();
if (activeStar.isNull()) {
QSvgRenderer render(QString(":star"));
QPixmap renderedStar(IMG_SIZE, IMG_SIZE);
@ -83,18 +85,17 @@ StarWidget::StarWidget(QWidget *parent, Qt::WindowFlags f) : QWidget(parent, f),
QPainter painter(&renderedStar);
render.render(&painter, QRectF(0, 0, IMG_SIZE, IMG_SIZE));
(*activeStar) = renderedStar;
activeStar = renderedStar.toImage();
}
if (!inactiveStar) {
inactiveStar = new QPixmap();
(*inactiveStar) = grayImage(activeStar);
if (inactiveStar.isNull()) {
inactiveStar = grayImage(activeStar);
}
setFocusPolicy(Qt::StrongFocus);
}
QPixmap StarWidget::grayImage(QPixmap *coloredImg)
QImage grayImage(const QImage& coloredImg)
{
QImage img = coloredImg->toImage();
QImage img = coloredImg;
for (int i = 0; i < img.width(); ++i) {
for (int j = 0; j < img.height(); ++j) {
QRgb rgb = img.pixel(i, j);
@ -107,7 +108,7 @@ QPixmap StarWidget::grayImage(QPixmap *coloredImg)
}
}
return QPixmap::fromImage(img);
return img;
}
QSize StarWidget::sizeHint() const

9
qt-ui/starwidget.h
View file

@ -17,8 +17,8 @@ public:
/*reimp*/ QSize sizeHint() const;
static QPixmap starActive();
static QPixmap starInactive();
static const QImage& starActive();
static const QImage& starInactive();
signals:
void valueChanged(int stars);
@ -39,9 +39,8 @@ private:
int current;
bool readOnly;
static QPixmap *activeStar;
static QPixmap *inactiveStar;
QPixmap grayImage(QPixmap *coloredImg);
static QImage activeStar;
static QImage inactiveStar;
};
#endif // STARWIDGET_H