Message: previous - next
Month: October 2014

Re: [trinity-devel] Improvements to the "Patches from GIT" web page

From: Michele Calgaro <michele.calgaro@...>
Date: Mon, 27 Oct 2014 11:56:50 +0900
On 2014/10/27 06:58 AM, Timothy Pearson wrote:
> 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

Tim, good work, thanks!
Just one thing I think we could improve: the links to [First Page] 
[Previous Page]  [Next Page] [Last Page] could be either moved or 
duplicated at the top of the page. If a user wants to quickly scroll 
pages, at this time he has to scroll at the end of each page (or 
alternatively modify the address manually). What do you think?

"Commit history" sounds good.


PS: by the way I am edging closer to complete bug 1859 (free time is 
again a rarity these days). I should be able to get it done by this week.