-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA224
The new "patches from GIT" page looks good
and so does the search field
at the bottom. Anyhow IMO there are still a few things to fix:
1) email addresses are shown in full, which is not that good being a
page available without any login.
It would be advisable to remove them, both from the committer column and
the Log message column (for signed-off commits).
While I don't fully agree with this (e.g. anyone can go to CGIT via the
"View" link and see all the Email addresses there, or even download the
repositories via GIT and retrieve all the Email addresses automatically
from that local copy) the Email addresses were wasting valuable space so I
went ahead and removed them. Publicly-known Email addresses are one of
the hazards of contributing to open source. ;-)
2) I believe the correct name of the committer column
should be "Author"
Fixed.
3) The page still displays with problems if the width
of the browser
window is small enough (the blue left side menu overlaps the page
contents)
Not sure I can do anything about this; E. Liddell would have to comment.
4) The search field would probably be better if it was
at the top, since
it does not take up much page space. At the moment we need to scroll the
whole page if we want to search for a particular commit
Reworked.
5) the 'page navigation' (the one saying
'Now viewing patches...') would
be better if it had links to the next and previous adjacent pages as in
most search engines (something like [1], [2], [3], .... to make it clear)
I reworked the navigation system; is it more functional now?
6) IMO, the name "Patches from GIT" is
misleading for new users, as I
already said in a previous email. Would be better to call it "GIT
Commits" or "Commit history" or something with "commit" in it
Opinions from the list are welcome here. I will probably go with "Commit
History" if no one else has a better suggestion.
Hope this is useful.
Cheers
Michele
Yes, thanks!
Tim
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
iFYEARELAAYFAlRNbmsACgkQLaxZSoRZrGEYOADfZYrnoSR4gyZ4TY54dd4Jg/GX
QBNsO5QCYwJ6HADgs0S7XsaI9VzWYPm1W1nHMexTTChlZXOO2eJqnw==
=ImLp
-----END PGP SIGNATURE-----