gitweb: Use light/dark for class names also in 'blame' view

Instead of using "light2" and "dark2" for class names in 'blame' view
(in place of "light" and "dark" classes in other places) to avoid
changing style on hover in 'blame' view while doing it for other views
(like 'shortlog'), use more advanced CSS, relying on the fact that
more specific selector wins.

While at it add a few comments to gitweb CSS file, and consolidate
some repeated info.

Signed-off-by: Jakub Narebski <redacted>
Signed-off-by: Junio C Hamano <redacted>

diff --git a/gitweb.css b/gitweb.css
index 10868fadd45112702267cb6fd92ee935458b997e6b26e48e7331dae09b2f3637..7789945d04e5e9c497dd52106647212efe31937af8939a5d55272c43b0bb0dcd 100644 (file)
--- a/gitweb.css
+++ b/gitweb.css
@@ -226,22 +226,25 @@ th {
        text-align: left;
 }
 
-tr.light:hover {
-       background-color: #edece6;
-}
-
-tr.dark {
-       background-color: #f6f6f0;
+/* do not change row style on hover for 'blame' view */
+tr.light,
+table.blame .light:hover {
+       background-color: #ffffff;
 }
 
-tr.dark2 {
+tr.dark,
+table.blame .dark:hover {
        background-color: #f6f6f0;
 }
 
+/* currently both use the same, but it can change */
+tr.light:hover,
 tr.dark:hover {
        background-color: #edece6;
 }
 
+/* boundary commits in 'blame' view */
+/* and commits without "previous" */
 tr.boundary td.sha1,
 tr.no-previous td.linenr {
        font-weight: bold;

diff --git a/gitweb.perl b/gitweb.perl
index e4cac4512397de7466452af99ba4e822d133b574c561509abc3502b5a5a2190c..5b78db5709e4eda8d794ffcc937216a2745476167013545774c23a88ed785a31 100755 (executable)
--- a/gitweb.perl
+++ b/gitweb.perl
@@ -4802,7 +4802,7 @@ sub git_blame {
        git_print_page_path($file_name, $ftype, $hash_base);
 
        # page body
-       my @rev_color = qw(light2 dark2);
+       my @rev_color = qw(light dark);
        my $num_colors = scalar(@rev_color);
        my $current_color = 0;
        my %metainfo = ();