Couleur QPushButton juste au survole

Version imprimable

J'ai 2 classes:
MM_Test qui contient ceci:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
MM_Test::MM_Test(QWidget *parent) : QDialog(parent) {
        mmw = new MM_Widget(this);
	but = new QPushButton(this);
	this->resize(300, 50);
	connect(but, SIGNAL(clicked()), this, SLOT(setColor()));
}
 
void MM_Test::setColor() {
	QColor *col = new QColor(255, 0, 0, 0);
	mmw->setColor(col);
 
}

MM_Widget qui dérive de QWidget

Code:

1
2
3
4
5
6
7
8
9
 
void MM_Widget::setColor(QColor *col) {
	if(col->isValid()) {
		*mm_color = *col;
		QPalette pal = colorButton->palette();
		pal.setColor(QPalette::Button, *mm_color);
		colorButton->setPalette(pal);
	}
}

La nouvelle couleur de colorButton n'apparait que lorsque je le survole avec ma souris. Si je clic sut but
Lorsque je clic sur mon bouton (colorButton), j'appelle ce code:

Code:

1
2
3
4
5
6
7
8
9
void MM_Widget::updateColor() {
	QColor col = QColorDialog::getColor(*mm_color, this);
	if(col.isValid()) {
		*mm_color = col;
		QPalette pal = colorButton->palette();
		pal.setColor(QPalette::Button, *mm_color);
		colorButton->setPalette(pal);
	}
}

et là, aucun soucis.

Où ai-je pêché? :?

12/09/2008, 17h09
yan

dsl,
mais j'ai rien compris.....

MM_Test contient un bouton but et un widget mmw
Le Widget contient lui-même un bouton colorButton et d'autres trucs.

Si je clic sur but, j'appelle mmw->setColor(QColor *col). Là, çà marche pas bien.

Si je clic sur colorButton, j'appelle updateColor(). Là, çà marche bien.

Dysfonctionnement:
La couleur du bouton n'est pas changée. Elle ne change que lorsque je survole le bouton avec ma souris.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#include "mm_test.h"
#include <QPushButton>
 
MM_Test::MM_Test(QWidget *parent) : QDialog(parent) {
	QVBoxLayout layout(this);
 
	mmw = new MM_Widget(this);
	but = new QPushButton(this);
	layout.addWidget(mmw);
	layout.addWidget(but);
	this->resize(300, 50);
	connect(but, SIGNAL(clicked()), this, SLOT(setColor()));
}
 
void MM_Test::setColor() {
	QColor *col = new QColor(255, 0, 0, 0);
	mmw->setColor(col);
 
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
#include <QtGui>
#include "MM_Widget.h"
 
MM_Widget::MM_Widget(QWidget *parent) : QWidget(parent) {
	mm_value = 0;
	mm_color = new QColor(255, 0, 0, 0);
	horizontalLayout = new QHBoxLayout(this);
	horizontalLayout->setContentsMargins(2, 2, 2, 2);
	checkBox = new QCheckBox("lalala", this);
	horizontalLayout->addWidget(checkBox);
	spinBox = new QSpinBox(this);
	spinBox->setMaximum(500);
	horizontalLayout->addWidget(spinBox);
	colorButton = new QPushButton("", this);
	horizontalLayout->addWidget(colorButton);
	colorButton->setMaximumHeight(20);
	spinBox->setEnabled(false);
	colorButton->setEnabled(false);
 
	// connections
	connect(colorButton, SIGNAL(clicked()), this, SLOT(updateColor()));
}
 
void MM_Widget::updateColor() {
	QColor col = QColorDialog::getColor(*mm_color, this);
	if(col.isValid()) {
		*mm_color = col;
		QPalette pal = colorButton->palette();
		pal.setColor(QPalette::Button, *mm_color);
		colorButton->setPalette(pal);
	}
}
 
 
void MM_Widget::setColor(QColor *col) {
	if(col->isValid()) {
		*mm_color = *col;
		QPalette pal = colorButton->palette();
		pal.setColor(QPalette::Button, *mm_color);
		colorButton->setPalette(pal);
	}
}
]

12/09/2008, 19h09
yan

De ce que j'ai compris,
quand tu appuie sur but, tu change la palette d'un element d'une autre widget.
Seulement, ca ne semble pas appeler un update de cette widget.
Ajoute colorButton->update() http://qt.developpez.com/doc/latest/qwidget.html#update
a la fin de setColor(QColor *col)
12/09/2008, 21h14
MacPro

salut, pourquoi tu n'utilise pas betement les feuilles de styles ? surtout que le QPushButton supporte le 'box model'

http://qt.developpez.com/doc/latest/...ng-qpushbutton

tu utilises hover pour changer la couleur lors du survol du bouton, puis pressed lorsqu'il est appuye et d'autres ...
13/09/2008, 15h27
Vespasien

1 pièce(s) jointe(s)

J'ai bien essayé d'ajouter colorButton->update() mais le comportement est toujours aussi déconcertant.

Capture du widget jointe:
MM_Widget contient: checkBox, spinBox et 'colorButton'
MM_Test est la fenêtre contenant MM_Widget + but

Pourquoi je n'utilise pas CSS?
Cette appli est un programme perso. Je refais en C++ une application que j'avais réalisé en JAVA il y a quelques années. Le but: dérouiller mon C++ et avoir de la programmation interessante à faire (au boulot, c'est pas souvent le cas).
CSS, je ne connais pas et çà ne m'intéresse pas. Mais, ici, la problématique est ailleurs de toute façon. :D
13/09/2008, 15h54
yan

Citation:

Envoyé par Vespasien

J'ai bien essayé d'ajouter colorButton->update() mais le comportement est toujours aussi déconcertant.

et si tu ne met juste update()??
13/09/2008, 17h59
droggo

Kao,

Quand on connecte un signal et un slot, il faut que qu'ils aient le même nombre de paramètres, et de même type.

Donc

Code:

connect(but, SIGNAL(clicked()), this, SLOT(setColor()));

pour appeler

Code:

void MM_Widget::setColor(QColor *col)

ne marche pas correctement, car en fait setColor utilise un pointeur qui n'est pas passé par le signal, et qui contient donc n'importe quoi.
13/09/2008, 18h48
Vespasien

Quand on connecte un signal et un slot, il faut que qu'ils aient le même nombre de paramètres, et de même type.
Ils ont le même nombre de paramètres, c'est à dire aucun. Puisque mon appel setColor() renvoi sur une fonction qui elle crée une QColor et appelle ensuite setColor(QColor *) de l'autre classe.

De toute façon, la couleur est bien transmise puisqu'au survole par la souris, elle s'affiche.

Surprenant:
Je choisi une couleur avec colorButton en cliquant dessus (bleu).
Je clic sur but et colorButton perd la couleur que je viens de lui donner (reprend la couleur grise initiale). Au survole, bien entendu, je trouve la nouvelle couleur (rouge).

Tous les update() n'ont rien changés.:(
14/09/2008, 10h56
yan

Es ce que tu pourrai faire un petit exemple compilable avec ton problème?
Pour tester.
C'est vraiment curieux.

C'est quel version de Qt? tu compile avec quoi?
14/09/2008, 11h02
Vespasien

1 pièce(s) jointe(s)

Justement, je compile actuellement la version 4.4.1 de Qt. Je travaillais avec Qt-4.4.0.
Environnement: Linux Slamd64

Puisque apparament, le bug n'est pas face au clavier c'est peut-être un bug mineur corrigé entre les 2 versions.

C-joint un zip contenant les 5 fichiers sources (main.cpp, MM_Test.cpp, MM_Test.h, MM_Widget.cpp, MM_Widget.h). Modifier BUTTON_HEIGHT, c'est une variable globale située dans un autre fichier .h.
14/09/2008, 12h20
yan
bon en faite, j'en suis au même point que toi.
En cherchant un peu sur internet, j'ai trouvé que le problème est dû au style. Mais ce n'est pas un bug...

Il faut donc utiliser un stylesheet.
http://qt.developpez.com/doc/latest/...ng-qpushbutton
Code:

1 2 3 QString style ="background-color: "+col.name (); colorButton->setStyleSheet(style);
Par contre, pourquoi tu utiliser des pointeurs sur QColor?????
14/09/2008, 12h58
droggo

Hao,

Citation:

Envoyé par Vespasien

Quand on connecte un signal et un slot, il faut que qu'ils aient le même nombre de paramètres, et de même type.
Ils ont le même nombre de paramètres, c'est à dire aucun. Puisque mon appel setColor() renvoi sur une fonction qui elle crée une QColor et appelle ensuite setColor(QColor *) de l'autre classe.

Désolé, j'avais lu un peu trop en diagonale. :)
14/09/2008, 16h33
Vespasien

Désolé, j'avais lu un peu trop en diagonale.
Je l'avais compris. :)

Citation:

Envoyé par Mongaulois

Par contre, pourquoi tu utiliser des pointeurs sur QColor?????

J'ai pas trop chercher, dans tous les cas le résultat est aussi surprenant.

J'ai tenté le coup avec les feuilles de style, çà fonctionne bien mais je me demandais comment faire pour la retrouver au lancement de l'application. Du coup, j'ai pas creusé. Effectivement name() satisfait parfaitement.

Je considère la chose comme résolue, bien que nous ayons contournés le mur et que cette surprise me laisse un gout bizarre.

MERCI à tous (même ceux qui ont lus en diagonale :chin:).