Just some optical improvements:
-Layout the Dialog after inserting values
-Better formatting of date and time
-added bytes to the filelength (18456 bytes)
[attachment deleted by admin]
According to the patch the date is shown using the current locale. That means those of us who have configured the system to show it as dd/mm/yyyy would get it like that. That was my concern.
Patch looks good to me. I have NOT applied it, though.
Here is an update:
i reverted my date formating changes, but i changed the format string, so that the values will be filled with 0 if they are less then 10.
old format string:
"%ld/%ld/%ld %ld:%ld:%ld"
new format string:
"%02ld/%02ld/%ld %02ld:%02ld:%02ld"
[attachment deleted by admin]
great, but I think the %ld Bytes should be _() since the word bytes could be translated into other languages.
On my OpenSuse 10.3 box (aged distribution ;-) ), I get the following for a timestamp (exiting CB code, the patch not applied yet) :
21/3/2010 14:140733193388067:22
Quote from: killerbot on March 28, 2010, 12:06:39 AM
great, but I think the %ld Bytes should be _() since the word bytes could be translated into other languages.
Yes your are right. I changed it.
for the other problem, i think the reason is, that we use ld, which means long int or unsigned long int.
instead we should use hd, wich means short int or unsigned short int, because wxDateTime_t is defined as:
typedef unsigned short wxDateTime_t;[attachment deleted by admin]
applied, and on my OpenSuse 10.3 box, I now get normal time stamps :-)
btw,
this patch solved bug 16288 on berlios
https://developer.berlios.de/bugs/?func=detailbug&bug_id=16288&group_id=5358 (https://developer.berlios.de/bugs/?func=detailbug&bug_id=16288&group_id=5358)
so it could be closed
greets GeO
done. Thanks for pointing this out.