Rendre darktable moins moche

[nicoauffray]

@jpg54 : logiquement ça va le permettre oui, comme avant.

@pascal : je viens de constater un bug. Lorsque je change le thème, la modification est instantanée mais pas pour toutes les balises CSS. Je n'ai pas regardé en détail, la majorité étant changée instantanément. La balise (et importante) que j'ai constaté est celle permettant de définir la couleur de fond autour de l'image dans la chambre noire :

Code :

@define-color darkroom_bg_color #333333;

Si sur un autre CSS, j'ai mis une autre couleur, la couleur ne change pas après modification du thème. Elle ne change qu'après redémarrage de darktable. Si ensuite, je reviens sur le CSS darktable par défaut, idem, cette couleur ne change qu'après redémarrage de darkable. Je n'ai pas remarqué d'autres balises impactées donc à priori toutes les autres balises importantes (les plus visibles) se modifient bien instantanément.

[pascal]

Si j'arrive a reproduire je verrais, il faut peut-être forcer un redraw de tous les widgets! Si je te donne un patch, tu pourrais tester?

[nicoauffray]

Si j'arrive a reproduire je verrais, il faut peut-être forcer un redraw de tous les widgets! Si je te donne un patch, tu pourrais tester?

Oui, je peux me faire une compil séparée (déjà fait).

[pascal]

Ok, voilà le diff alors, très simple. Juste une ligne à ajouter dans preferences.c:

Code :

diff --git a/src/gui/preferences.c b/src/gui/preferences.c
index 8127202a4..2786c15bd 100644
--- a/src/gui/preferences.c
+++ b/src/gui/preferences.c
@@ -178,6 +178,7 @@ static void theme_callback(GtkWidget *widget, gpointer user_data)
  gchar *i = g_strrstr(theme, ".");
  if(i) *i = '\0';
  dt_gui_load_theme(theme);
+  dt_control_queue_redraw();
}

///////////// gui language selection

[nicoauffray]

Ok, voilà le diff alors, très simple. Juste une ligne à ajouter dans preferences.c:

Code :

diff --git a/src/gui/preferences.c b/src/gui/preferences.c
index 8127202a4..2786c15bd 100644
--- a/src/gui/preferences.c
+++ b/src/gui/preferences.c
@@ -178,6 +178,7 @@ static void theme_callback(GtkWidget *widget, gpointer user_data)
  gchar *i = g_strrstr(theme, ".");
  if(i) *i = '\0';
  dt_gui_load_theme(theme);
+  dt_control_queue_redraw();
}

///////////// gui language selection

Cette ligne ne change rien. Je me suis bien assuré d'avoir la dernière révision. Et je dois toujours redémarrer dt pour que le fond de la chambre noire (et de la table lumineuse) se modifie.
J'ai beau regardé attentivement l'interface et mes modifs faites, je ne vois vraiment que ça qui ne se modifie qu'après redémarrage. Il y a sûrement un truc à creuser du coup du côté du code GTK qui gère cette partie (fond table lumineuse et chambre noire).

[pascal]

@nicoauffray : ça devrait marcher maintenant! peux-tu tester sur master?

[nicoauffray]

@nicoauffray : ça devrait marcher maintenant! peux-tu tester sur master?

Well done ! Ça fonctionne parfaitement.

[pascal]

Cool! Et hop encore un bug en moins

[Roger]

Je ne suis pas dev donc je vais sans doute dire une c***rie.

Est ce qu'il ne serait pas possible, à la prochaine mise à jours, de créer le dossier "Thèmes" automatiquement, d'y mettre/déplacer le CSS par défaut dedans et de faire un joli pop-up à l'ouverture pour prévenir l'utilisateur qu'il ne lui reste plus qu'a déplacer son CSS personnalisé et/ou les thèmes téléchargés dans ce fameux dossier?

[pascal]

@Roger: regarde ma réponse plus haut. le CSS par défaut n'a pas besoin d'être déplacé il est livré avec dt.

Si tu parles de celui par défaut pour l'utilisateur. Pas simple, encore une fois relis mon message plus haut.