mifr/subsurface
1
0
Fork 0
mirror of https://github.com/subsurface/subsurface.git synced 2025-02-19 22:16:15 +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
Download patch file Download diff file Expand all files Collapse all files

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