Suggestions for 1.7.4 and higher

More
11 years 7 months ago - 11 years 7 months ago #12463 by MrX1980
Suggestions for future versions based on 1.7.3

A)
header.php / Line 7-9:
Code:
old: include("gbclass.php"); include("config.php"); include("language/$default_language"); new: require_once("gbclass.php"); require_once("config.php"); require_once("language/$default_language");

header.php / Line 15:
Code:
old: <title>Powered by DigiOz Guestbook Version 1.7.2</title> new: <title>Powered by DigiOz Guestbook</title>

header.php / Line 18:
Code:
old: <body bgcolor="#FFFFFF" text="#000000" link="#8080ff" vlink="#0000ff" alink="#ffff00"><center> new: <body><center>
header.php / Line 32+33:
Code:
old: <td><font size="1">Enter Search <b>WORD</b>:</font></td> <td align=left><input type="text" name="search_term" size=20 maxlength=50><input type=submit value="Find Now!"></td> new: <td><font size="1"><?php echo $Enter_Search_Word ?></font></td> <td align=left><input type="text" name="search_term" size=20 maxlength=50><input type=submit value="<?php echo $Find_Now ?>"></td>


B) replace language_english.php with language.php and remove/delete language.php

C)
language_english.php
add
Code:
$Enter_Search_Word = "Enter Search WORD:"; $Find_Now = "Find Now!"; $Verify = "Verify"; $Bold = "Bold"; $Underline = "Underline"; $Italic = "Italic"; $Center = "Center";

language_german.php
add
Code:
$Enter_Search_Word = "Suche nach:"; $Find_Now = "Suche"; $Verify = "Abgleich"; $Bold = "Fett"; $Underline = "Unterstrichen"; $Italic = "Kursiv"; $Center = "Zentriert";

D) Many messages are not translated via language_...php

E) Don't use numbers please. Give it meaningful names.
$error1 = $Error_Long_Name
$error2 = $Error_Long_Email
$error3 = $Error_Invalid_Email
...

F)
guestbook.php / Line 66-69
Code:
old: <td><input type="button" class="button" value=" Bold " style="font-weight:bold; width: 50px" onClick="javascript:bbcode('[b] [/b]')"></td> <td><input type="button" class="button" value=" Underline " style="font-weight:bold; width: 80px" onClick="javascript:bbcode('[u] [/u]')"></td> <td><input type="button" class="button" value=" Italic " style="font-weight:bold; width: 50px" onClick="javascript:bbcode('[i] [/i]')"></td> <td><input type="button" class="button" value=" Center " style="font-weight:bold; width: 70px" onClick="javascript:bbcode('[center] [/center]')"></td> new: <td><input type="button" class="button" value=" <?php echo $Bold; ?> " style="font-weight:bold; width: 50px" onClick="javascript:bbcode('[b] [/b]')"></td> <td><input type="button" class="button" value=" <?php echo $Underline; ?> " style="font-weight:bold; width: 100px" onClick="javascript:bbcode('[u] [/u]')"></td> <td><input type="button" class="button" value=" <?php echo $Italic; ?> " style="font-weight:bold; width: 50px" onClick="javascript:bbcode('[i] [/i]')"></td> <td><input type="button" class="button" value=" <?php echo $Center; ?> " style="font-weight:bold; width: 70px" onClick="javascript:bbcode('[center] [/center]')"></td>

guestbook.php / Line 94
Code:
old: echo '<tr><td><p><b><font size="2">Verify:</font></b></p></td><td><input type="text" name="txtNumber" size="20" value=""><img src="random.php"></td></tr>'; new: echo '<tr><td><p><b><font size="2">'.$Verify.'</font></b></p></td><td><input type="text" name="txtNumber" size="20" value=""><img src="random.php"></td></tr>';

G)
add.php / Line 74
Code:
$date = date("D m/j/y g:iA");
move it to language_....php (for regional preferences)

H)
list.php / Line 30+31
Code:
// Setting the default values for number of records per page ------------------------- $perpage = 10;
move it to config.php

I) I see to many <font ...> tags in the code. Use more the style.css


... to be continued.
Last edit: 11 years 7 months ago by MrX1980.

Please Log in or Create an account to join the conversation.

More
11 years 7 months ago #12467 by Pete
Thank you very much for your feedback. Your suggestion about "Powered by" text has already been implemented in version 1.7.3. The remaining suggestions are excellent suggestions and will be implemented for the next release.

Pete

DigiOz Webmaster
www.digioz.com

Please Log in or Create an account to join the conversation.

More
11 years 7 months ago #12472 by MrX1980

Pete wrote: Your suggestion about "Powered by" text has already been implemented in version 1.7.3.

Not on all "places", it was only done for the footer.php but not the header.php :-)

Do you have a roadmap or something else for new versions ?

Please Log in or Create an account to join the conversation.

More
11 years 7 months ago #12473 by Pete
Good catch. I will change the header for the next release.

As far as release schedule, right now I am the only one working on the project and there is no set release schedule. We had a team of 5 at one point on this project, but over time they lost interest and moved on. You are welcome to join if you wish to contribute your time towards this open source project. Let me know if you are interested.

Pete

DigiOz Webmaster
www.digioz.com

Please Log in or Create an account to join the conversation.

Time to create page: 0.119 seconds
Powered by Kunena Forum