[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Property box



Hi everyone.

I have uploaded a diff to implement a property box in Dr. Genius. 
You can find the diff here :
 
http://www.celorio.com/gnome/property_box.diff

Here are some issues that I need to let you know about :

1) Please keep in mind that this is my first Linux/GNOME/GTK/Dr-GENIUS
code,
( Beer !!!!! ) so any comments will be greatly appreciated. This has
been a
great learning mini-mini-hack for me. 

2) Issues when creating the diff
	I could not find a way to eliminate the :
	"only in .." messages when I created the diff.
	I don't know if this will have any effect when you patch
	the code. If it does please let me know and I will
	make the diff again.

	I made a copy of the directory before coding
	and the diff command I used was :
	"diff -ru dr-genius/ dr-genius.original/ > property_box.diff"

3) I just implemented one option, just in case you think
I need to do something different. After this patch gets
committed to the cvs server I will continue with the other
options. The option was Calculator/set_max_digits

4) The code was included in drgenius_menus.c which I guess is not
the right place to put it. I did not know where this code should be
or if this code should be in a file by itself. Because of that, I
included
the following comments.
/* -------- Property box prototypes ----------*/
/* --------- END property box proto ....
/* -------- Property box variables ----------*/
/* --------- END  ....
/* -------- Property box code ----------*/
/* --------- END  ....

so that it can be relocated easily.

5) In order for this to work there has to be a Calculator open, is
NOT that it will crash, only that it will not affect the variables.
I will work on this latter.

Thank You
Chema Celorio