Michal Čihař wrote:
Hi
On Tue 4. 10. 2005 10:13, Sebastian Mendel wrote:
i would like to commit this into CVS, if no one
is against.
I just quickly looked at the patch, My comments follow. The all do not
address new code, however when the code is being touched, it could be
cleaned up :-). Those are just minor issues, generally I like it.
- do not hardcode color in global css
i will take a look at this,
- do not change error reporting
fixed lokaly, forgot to remove before creating the patch file
- you can rely on fact that all cfg parameters are
set, this includes
PmaAbsoluteUri
changed in index.php and left.php
- I'm absolutely not sure about browser dependant
css you removed from
theme
tested with IE 6, 5.5, 5, Opera, Firefox
and complies with my experience with CSS
- did you intentionally remove +/- signs in front of
expanded/non
expanded database in left frame?
only if only one db is available, so this db is allways expanded and
needs no +/-, or is this missing anywhere else?
- tables and databases are now printed same colour in
left frame, I
think different colour was better
i will take a look at this
- frameset contains urls like
left.php?&lang=cs-utf-8, there is
extra &
for sure? i cant reproduce
--
Sebastian Mendel
www.sebastianmendel.de
www.sf.net/projects/phpdatetime |
www.sf.net/projects/phptimesheet