It looks nice, but you have all values hardcoded. Some people use .php3 files instead of the "normal" .php extension. Others have differen't languages installed. Please also give us a text link.
The paths don't have a / at the beginning. It's not
Same goes for other paths inside the code.
src="/phpBB/templates/subSilver/images/icon_mini_search.gif" width="12" height="13" border="0" align="top" alt="Search"
It has unnessecairy spaces there, "Search" is hardcoded and the path should be something like this:
Code: Select all
It should ALWAYS be relative to the phpBB root, and not to the webservers root.
There already is a MOD that does nearly the same as this one: Search box
. You could post it as an addon / enhancement for that
Don't take it personal, it's not bad for a first MOD! I just want to give you some tipps to improve!