Discussion :

[Vespasien]

J'ai 2 classes:
MM_Test qui contient ceci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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é?

[yan]

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

[Vespasien]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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);
	}
}
]

[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)

[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 ...

[Vespasien]

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.

[yan]

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

et si tu ne met juste update()??

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

pour appeler

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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.

[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?

[Vespasien]

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.

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
QString style ="background-color: "+col.name ();
colorButton->setStyleSheet(style);

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

[droggo]

Hao,

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.

[Vespasien]

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

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 ).