[grisbi-devel] [PATCH 05/12] Rewriting the toolbar of scheduler list

[Pierre Biava]

Rémi Cardona écrivait le 05/03/2013 07:26 :
> Globalement ok.
>
> 1) pourquoi ne pas profiter de la réécriture pour passer à GtkUIManager
> voire même à GtkBuilder?
>

J'utiliserai GtkUIManager lorsque je mettrai à jour le menu général de 
grisbi afin d'utiliser les GtkActions. Je ne pense pas que ce soit un 
gros travail cette mise à jour,

Quand à GtkBuilder ça ne me parait pas souple pour ça.

> 2) quelques commentaires plus bas.
>
> Rémi
>
> Le lundi 04 mars 2013 à 21:19 +0100, pbiava a écrit :

>> +    frame = gtk_frame_new ( NULL );
>> +    gtk_box_pack_start ( GTK_BOX ( vbox ), frame, FALSE, FALSE, 0 );
>
> Pourquoi créer une frame ici? Est-elle vraiment nécessaire? Ne peut-on
> pas directement mettre la GtkToolbar dans la VBox?

La GtkFrame sert juste à faire une rupture dans le design de la fenêtre 
donc juste cosmétique.

>> -void gsb_gui_update_scheduler_toolbar ( void )
>> +void gsb_gui_update_scheduler_toolbar ( gint toolbar_style )
>
> Vu ce qu'il reste de la fonction et son rôle, je propose de la renommer
> en gsb_gui_scheduler_toolbar_set_style(). Ce n'est pas plus court, mais
> ça dit bien ce que fait la fonction, contrairement au nom actuel.

Ok pour renommer toutes ces fonction.

Cordialement.

-- 

A+

Pierre Biava