Re: [PATCH 3/4] gitweb: show notes in log

On Sat, 6 Feb 2010, Giuseppe Bilotta wrote:

2010/2/6 Jakub Narebski [off-list ref]:

quoted

On Thu, 4 Feb 2010, Giuseppe Bilotta wrote:

quoted

The notes are shown in full to the left of the log message.

Thats all good if you have wide (high resolution) screen, and your
project follows common commit message conventions of keeping lines in
commit message no longer than at most 80 characters, and you don't need
to use large size fonts.

What happens if screen size is too small to contain both commit message
and notes?  Does it do the sensible thing of putting notes _below_
commit message in such situation?  I do not know CSS+HTML enogh to
answer this question myself.

The CSS forces the width of the notes div at 150px, which is the
amount left to the left of the commit message. This means that notes
will line-wrap, but they will not shift the text.

[...]

quoted

quoted

+             print "$notes\n";
              print "<div class=\"log_body\">\n";
              git_print_log($co{'comment'}, -final_empty_line=> 1);
              print "</div>\n";

With respect to the question about what happens if the screen is not
wide enough, shouldn't notes be put in HTML source below body (commit
message)?

As I mentioned, notes width is fixed at the amount of the whitespace
to the left of the log, so this should not be an issue. Additionally,
putting notes below makes it much harder to let them float to the left
of the log body.

Perhaps the log body should be floated to the right, instead of notes
being floated to the left, so that when screen width is to narrow for
both commit message and notes, notes would be put below commit message.

A question how to create styles using HTML elements and CSS styling
to get side-by-side with one below other as fallback can be asked
on http://stackoverflow.com, or perhaps even better on http://doctype.com/

-- 
Jakub Narebski
Poland