git@vger.kernel.org list mirror (unofficial, one of many)
 help / color / mirror / code / Atom feed
* [PATCH 3/4] gitweb: omit the repository owner when it is unset
  2013-07-14 23:49 ` [PATCH] gitweb: Ensure OPML text fits inside its box Jonathan Nieder
@ 2013-07-02 16:24   ` Tony Finch
  2013-07-09 16:43   ` [PATCH 1/4] gitweb: Ensure OPML text fits inside its box Tony Finch
                     ` (5 subsequent siblings)
  6 siblings, 0 replies; 20+ messages in thread
From: Tony Finch @ 2013-07-02 16:24 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski, Jonathan Nieder

On the repository summary page, leave the whole owner line out if
the repo does not have an owner, rather than displaying a labelled
empty field..

Signed-off-by: Tony Finch <dot@dotat.at>
---
 gitweb/gitweb.perl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 8d69ada..c029b98 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -6463,7 +6463,7 @@ sub git_summary {
 	print "<div class=\"title\">&nbsp;</div>\n";
 	print "<table class=\"projects_list\">\n" .
 	      "<tr id=\"metadata_desc\"><td>description</td><td>" . esc_html($descr) . "</td></tr>\n";
-        unless ($omit_owner) {
+        if ($owner and not $omit_owner) {
 	        print  "<tr id=\"metadata_owner\"><td>owner</td><td>" . esc_html($owner) . "</td></tr>\n";
         }
 	if (defined $cd{'rfc2822'}) {
-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH] gitweb: Ensure OPML text fits inside its box.
@ 2013-07-09 16:43 Tony Finch
  2013-07-09 17:08 ` [PATCH] gitweb: vertically centre contents of page footer Tony Finch
  2013-07-14 23:49 ` [PATCH] gitweb: Ensure OPML text fits inside its box Jonathan Nieder
  0 siblings, 2 replies; 20+ messages in thread
From: Tony Finch @ 2013-07-09 16:43 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski

The rss_logo CSS style has a fixed width which is too narrow for
the string "OPML". Replace the fixed width with horizontal padding
so the text fits with nice margins.
---

For before/after examples, see http://dotat.at/cgi/git (overflow)
and https://git.csx.cam.ac.uk/x/ucs/ (padded).

 gitweb/static/gitweb.css | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/gitweb/static/gitweb.css b/gitweb/static/gitweb.css
index cb86d2d..a869be1 100644
--- a/gitweb/static/gitweb.css
+++ b/gitweb/static/gitweb.css
@@ -548,8 +548,7 @@ a.linenr {
 
 a.rss_logo {
 	float: right;
-	padding: 3px 0px;
-	width: 35px;
+	padding: 3px 5px;
 	line-height: 10px;
 	border: 1px solid;
 	border-color: #fcc7a5 #7d3302 #3e1a01 #ff954e;
-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH 1/4] gitweb: Ensure OPML text fits inside its box.
  2013-07-14 23:49 ` [PATCH] gitweb: Ensure OPML text fits inside its box Jonathan Nieder
  2013-07-02 16:24   ` [PATCH 3/4] gitweb: omit the repository owner when it is unset Tony Finch
@ 2013-07-09 16:43   ` Tony Finch
  2013-07-09 17:08   ` [PATCH 2/4] gitweb: vertically centre contents of page footer Tony Finch
                     ` (4 subsequent siblings)
  6 siblings, 0 replies; 20+ messages in thread
From: Tony Finch @ 2013-07-09 16:43 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski, Jonathan Nieder

The rss_logo CSS style has a fixed width which is too narrow for
the string "OPML". Replace the fixed width with horizontal padding
so the text fits with nice margins.

Signed-off-by: Tony Finch <dot@dotat.at>
---
 gitweb/static/gitweb.css | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/gitweb/static/gitweb.css b/gitweb/static/gitweb.css
index cb86d2d..a869be1 100644
--- a/gitweb/static/gitweb.css
+++ b/gitweb/static/gitweb.css
@@ -548,8 +548,7 @@ a.linenr {
 
 a.rss_logo {
 	float: right;
-	padding: 3px 0px;
-	width: 35px;
+	padding: 3px 5px;
 	line-height: 10px;
 	border: 1px solid;
 	border-color: #fcc7a5 #7d3302 #3e1a01 #ff954e;
-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH] gitweb: vertically centre contents of page footer
  2013-07-09 16:43 [PATCH] gitweb: Ensure OPML text fits inside its box Tony Finch
@ 2013-07-09 17:08 ` Tony Finch
  2013-07-14 23:49 ` [PATCH] gitweb: Ensure OPML text fits inside its box Jonathan Nieder
  1 sibling, 0 replies; 20+ messages in thread
From: Tony Finch @ 2013-07-09 17:08 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski

---
 gitweb/static/gitweb.css | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gitweb/static/gitweb.css b/gitweb/static/gitweb.css
index a869be1..3b4d833 100644
--- a/gitweb/static/gitweb.css
+++ b/gitweb/static/gitweb.css
@@ -68,12 +68,13 @@ div.page_path {
 }
 
 div.page_footer {
-	height: 17px;
+	height: 22px;
 	padding: 4px 8px;
 	background-color: #d9d8d1;
 }
 
 div.page_footer_text {
+	line-height: 22px;
 	float: left;
 	color: #555555;
 	font-style: italic;
-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH 2/4] gitweb: vertically centre contents of page footer
  2013-07-14 23:49 ` [PATCH] gitweb: Ensure OPML text fits inside its box Jonathan Nieder
  2013-07-02 16:24   ` [PATCH 3/4] gitweb: omit the repository owner when it is unset Tony Finch
  2013-07-09 16:43   ` [PATCH 1/4] gitweb: Ensure OPML text fits inside its box Tony Finch
@ 2013-07-09 17:08   ` Tony Finch
  2013-07-09 17:15   ` [PATCH 4/4] gitweb: make search help link less ugly Tony Finch
                     ` (3 subsequent siblings)
  6 siblings, 0 replies; 20+ messages in thread
From: Tony Finch @ 2013-07-09 17:08 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski, Jonathan Nieder

Signed-off-by: Tony Finch <dot@dotat.at>
---
 gitweb/static/gitweb.css | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gitweb/static/gitweb.css b/gitweb/static/gitweb.css
index a869be1..3b4d833 100644
--- a/gitweb/static/gitweb.css
+++ b/gitweb/static/gitweb.css
@@ -68,12 +68,13 @@ div.page_path {
 }
 
 div.page_footer {
-	height: 17px;
+	height: 22px;
 	padding: 4px 8px;
 	background-color: #d9d8d1;
 }
 
 div.page_footer_text {
+	line-height: 22px;
 	float: left;
 	color: #555555;
 	font-style: italic;
-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH 4/4] gitweb: make search help link less ugly
  2013-07-14 23:49 ` [PATCH] gitweb: Ensure OPML text fits inside its box Jonathan Nieder
                     ` (2 preceding siblings ...)
  2013-07-09 17:08   ` [PATCH 2/4] gitweb: vertically centre contents of page footer Tony Finch
@ 2013-07-09 17:15   ` Tony Finch
  2013-07-15  9:15   ` [PATCH] gitweb: Ensure OPML text fits inside its box Tony Finch
                     ` (2 subsequent siblings)
  6 siblings, 0 replies; 20+ messages in thread
From: Tony Finch @ 2013-07-09 17:15 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski, Jonathan Nieder

The search help link was a superscript question mark right next to
a drop-down menu, which looks misaligned and is a cramped and
awkward click target. Remove the superscript tags and add some
spacing to fix these nits. Add a title attribute to provide an
explanatory mouseover.

Signed-off-by: Tony Finch <dot@dotat.at>
---
 gitweb/gitweb.perl | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index c029b98..874c948 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -4029,9 +4029,9 @@ sub print_search_form {
 	      $cgi->input({-name=>"a", -value=>"search", -type=>"hidden"}) . "\n" .
 	      $cgi->input({-name=>"h", -value=>$search_hash, -type=>"hidden"}) . "\n" .
 	      $cgi->popup_menu(-name => 'st', -default => 'commit',
-	                       -values => ['commit', 'grep', 'author', 'committer', 'pickaxe']) .
-	      $cgi->sup($cgi->a({-href => href(action=>"search_help")}, "?")) .
-	      " search:\n",
+			       -values => ['commit', 'grep', 'author', 'committer', 'pickaxe']) .
+	      " " . $cgi->a({-href => href(action=>"search_help"),
+			     -title => "search help" }, "?") . " search:\n",
 	      $cgi->textfield(-name => "s", -value => $searchtext, -override => 1) . "\n" .
 	      "<span title=\"Extended regular expression\">" .
 	      $cgi->checkbox(-name => 'sr', -value => 1, -label => 're',
-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH] gitweb: Ensure OPML text fits inside its box.
  2013-07-09 16:43 [PATCH] gitweb: Ensure OPML text fits inside its box Tony Finch
  2013-07-09 17:08 ` [PATCH] gitweb: vertically centre contents of page footer Tony Finch
@ 2013-07-14 23:49 ` Jonathan Nieder
  2013-07-02 16:24   ` [PATCH 3/4] gitweb: omit the repository owner when it is unset Tony Finch
                     ` (6 more replies)
  1 sibling, 7 replies; 20+ messages in thread
From: Jonathan Nieder @ 2013-07-14 23:49 UTC (permalink / raw)
  To: Tony Finch; +Cc: git, Jakub Narębski

Tony Finch wrote:

> The rss_logo CSS style has a fixed width which is too narrow for
> the string "OPML". Replace the fixed width with horizontal padding
> so the text fits with nice margins.

Sounds sensible.  Can we have your sign-off?  (Likewise for the next
patch.)

Thanks,
Jonathan

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH] gitweb: Ensure OPML text fits inside its box.
  2013-07-14 23:49 ` [PATCH] gitweb: Ensure OPML text fits inside its box Jonathan Nieder
                     ` (3 preceding siblings ...)
  2013-07-09 17:15   ` [PATCH 4/4] gitweb: make search help link less ugly Tony Finch
@ 2013-07-15  9:15   ` Tony Finch
       [not found]   ` <E1Uyeth-0003TF-Kx@hermes-2.csi.cam.ac.uk>
       [not found]   ` <E1Uyete-0003Sv-OI@hermes-2.csi.cam.ac.uk>
  6 siblings, 0 replies; 20+ messages in thread
From: Tony Finch @ 2013-07-15  9:15 UTC (permalink / raw)
  To: Jonathan Nieder; +Cc: git, Jakub Narębski

Jonathan Nieder <jrnieder@gmail.com> wrote:
>
> Sounds sensible.  Can we have your sign-off?  (Likewise for the next
> patch.)

Doh! Thanks for looking at the patches. I'll post revised versions, plus a
couple more tweaks.

Tony.
-- 
f.anthony.n.finch  <dot@dotat.at>  http://dotat.at/
Forties, Cromarty: East, veering southeast, 4 or 5, occasionally 6 at first.
Rough, becoming slight or moderate. Showers, rain at first. Moderate or good,
occasionally poor at first.

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 4/4] gitweb: make search help link less ugly
       [not found]   ` <E1Uyeth-0003TF-Kx@hermes-2.csi.cam.ac.uk>
@ 2013-08-03 21:37     ` Jakub Narębski
  2013-08-05 11:19       ` Tony Finch
  0 siblings, 1 reply; 20+ messages in thread
From: Jakub Narębski @ 2013-08-03 21:37 UTC (permalink / raw)
  To: Tony Finch; +Cc: git, Jonathan Nieder

On Tue, Jul 9, 2013 at 7:15 PM, Tony Finch <dot@dotat.at> wrote:

> The search help link was a superscript question mark right next to
> a drop-down menu, which looks misaligned

I think the idea was to simulate footnote explaining search terms
(I think, I am not the author of this feature)...

>                                                                and is a cramped and
> awkward click target. Remove the superscript tags and add some
> spacing to fix these nits. Add a title attribute to provide an
> explanatory mouseover.

... but I agree that it makes for poor UI.

>
> Signed-off-by: Tony Finch <dot@dotat.at>
> ---
>  gitweb/gitweb.perl | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index c029b98..874c948 100755
> --- a/gitweb/gitweb.perl
> +++ b/gitweb/gitweb.perl
> @@ -4029,9 +4029,9 @@ sub print_search_form {
>               $cgi->input({-name=>"a", -value=>"search", -type=>"hidden"}) . "\n" .
>               $cgi->input({-name=>"h", -value=>$search_hash, -type=>"hidden"}) . "\n" .
>               $cgi->popup_menu(-name => 'st', -default => 'commit',
> -                              -values => ['commit', 'grep', 'author', 'committer', 'pickaxe']) .
> -             $cgi->sup($cgi->a({-href => href(action=>"search_help")}, "?")) .
> -             " search:\n",
> +                              -values => ['commit', 'grep', 'author', 'committer', 'pickaxe']) .

Nb. what changed here (in line above)?

> +             " " . $cgi->a({-href => href(action=>"search_help"),
> +                            -title => "search help" }, "?") . " search:\n",
>               $cgi->textfield(-name => "s", -value => $searchtext, -override => 1) . "\n" .
>               "<span title=\"Extended regular expression\">" .
>               $cgi->checkbox(-name => 'sr', -value => 1, -label => 're',
> --
> 1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 3/4] gitweb: omit the repository owner when it is unset
       [not found]   ` <E1Uyete-0003Sv-OI@hermes-2.csi.cam.ac.uk>
@ 2013-08-03 21:40     ` Jakub Narębski
  2013-08-05 11:36       ` Tony Finch
  0 siblings, 1 reply; 20+ messages in thread
From: Jakub Narębski @ 2013-08-03 21:40 UTC (permalink / raw)
  To: Tony Finch; +Cc: git, Jonathan Nieder

On Tue, Jul 2, 2013 at 6:24 PM, Tony Finch <dot@dotat.at> wrote:

> On the repository summary page, leave the whole owner line out if
> the repo does not have an owner, rather than displaying a labelled
> empty field..

Note that if $omit_owner is true, whole _column_ is skipped.

Is removing cell (instead of leaving it empty) and relying on browser
treating nonexistent cell correctly a good idea, that I do not know.
Does it looks better?  Does it looks better in all web browsers?

> Signed-off-by: Tony Finch <dot@dotat.at>
> ---
>  gitweb/gitweb.perl | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> index 8d69ada..c029b98 100755
> --- a/gitweb/gitweb.perl
> +++ b/gitweb/gitweb.perl
> @@ -6463,7 +6463,7 @@ sub git_summary {
>         print "<div class=\"title\">&nbsp;</div>\n";
>         print "<table class=\"projects_list\">\n" .
>               "<tr id=\"metadata_desc\"><td>description</td><td>" . esc_html($descr) . "</td></tr>\n";
> -        unless ($omit_owner) {
> +        if ($owner and not $omit_owner) {
>                 print  "<tr id=\"metadata_owner\"><td>owner</td><td>" . esc_html($owner) . "</td></tr>\n";
>          }
>         if (defined $cd{'rfc2822'}) {
> --
> 1.8.3.1.605.g85318f5
>



-- 
Jakub Narebski

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 4/4] gitweb: make search help link less ugly
  2013-08-03 21:37     ` [PATCH 4/4] gitweb: make search help link less ugly Jakub Narębski
@ 2013-08-05 11:19       ` Tony Finch
  0 siblings, 0 replies; 20+ messages in thread
From: Tony Finch @ 2013-08-05 11:19 UTC (permalink / raw)
  To: Jakub Narębski; +Cc: git, Jonathan Nieder, Tony Finch

[-- Attachment #1: Type: TEXT/PLAIN, Size: 609 bytes --]

Jakub Narębski <jnareb@gmail.com> wrote:

> > -                              -values => ['commit', 'grep', 'author', 'committer', 'pickaxe']) .
> > +                              -values => ['commit', 'grep', 'author', 'committer', 'pickaxe']) .
>
> Nb. what changed here (in line above)?

Whoops, tab damage. I will re-roll. Thanks for the review.

Tony.
-- 
f.anthony.n.finch  <dot@dotat.at>  http://dotat.at/
Forties, Cromarty: East, veering southeast, 4 or 5, occasionally 6 at first.
Rough, becoming slight or moderate. Showers, rain at first. Moderate or good,
occasionally poor at first.

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH 3/4] gitweb: omit the repository owner when it is unset
  2013-08-03 21:40     ` [PATCH 3/4] gitweb: omit the repository owner when it is unset Jakub Narębski
@ 2013-08-05 11:36       ` Tony Finch
  2013-08-20 17:22         ` [PATCH v4 0/4] Four small gitweb tweaks Tony Finch
  0 siblings, 1 reply; 20+ messages in thread
From: Tony Finch @ 2013-08-05 11:36 UTC (permalink / raw)
  To: Jakub Narębski; +Cc: git, Jonathan Nieder

[-- Attachment #1: Type: TEXT/PLAIN, Size: 903 bytes --]

Jakub Narębski <jnareb@gmail.com> wrote:
> On Tue, Jul 2, 2013 at 6:24 PM, Tony Finch <dot@dotat.at> wrote:
>
> > On the repository summary page, leave the whole owner line out if
> > the repo does not have an owner, rather than displaying a labelled
> > empty field..
>
> Note that if $omit_owner is true, whole _column_ is skipped.

There are two places where the owner is displayed: on the list of
projects, and on each project's summary page. This change affects the
summary page (where it removes a row, not a column) and it leaves the
projects list alone. I'll make that clearer in the commit message (and fix
the extraneous dot).

Tony.
-- 
f.anthony.n.finch  <dot@dotat.at>  http://dotat.at/
Forties, Cromarty: East, veering southeast, 4 or 5, occasionally 6 at first.
Rough, becoming slight or moderate. Showers, rain at first. Moderate or good,
occasionally poor at first.

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH v4 1/4] gitweb: Ensure OPML text fits inside its box.
  2013-08-20 17:22         ` [PATCH v4 0/4] Four small gitweb tweaks Tony Finch
@ 2013-08-20 16:59           ` Tony Finch
  2013-08-20 19:52             ` Junio C Hamano
  2013-08-20 16:59           ` [PATCH v4 2/4] gitweb: vertically centre contents of page footer Tony Finch
                             ` (2 subsequent siblings)
  3 siblings, 1 reply; 20+ messages in thread
From: Tony Finch @ 2013-08-20 16:59 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski, Jonathan Nieder

The rss_logo CSS style has a fixed width which is too narrow for
the string "OPML". Replace the fixed width with horizontal padding
so the text fits with nice margins.

Signed-off-by: Tony Finch <dot@dotat.at>
---
 gitweb/static/gitweb.css | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/gitweb/static/gitweb.css b/gitweb/static/gitweb.css
index cb86d2d..a869be1 100644
--- a/gitweb/static/gitweb.css
+++ b/gitweb/static/gitweb.css
@@ -548,8 +548,7 @@ a.linenr {
 
 a.rss_logo {
 	float: right;
-	padding: 3px 0px;
-	width: 35px;
+	padding: 3px 5px;
 	line-height: 10px;
 	border: 1px solid;
 	border-color: #fcc7a5 #7d3302 #3e1a01 #ff954e;
-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH v4 2/4] gitweb: vertically centre contents of page footer
  2013-08-20 17:22         ` [PATCH v4 0/4] Four small gitweb tweaks Tony Finch
  2013-08-20 16:59           ` [PATCH v4 1/4] gitweb: Ensure OPML text fits inside its box Tony Finch
@ 2013-08-20 16:59           ` Tony Finch
  2013-08-20 19:53             ` Junio C Hamano
  2013-08-20 16:59           ` [PATCH v4 3/4] gitweb: omit the repository owner when it is unset Tony Finch
  2013-08-20 16:59           ` [PATCH v4 4/4] gitweb: make search help link less ugly Tony Finch
  3 siblings, 1 reply; 20+ messages in thread
From: Tony Finch @ 2013-08-20 16:59 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski, Jonathan Nieder

Signed-off-by: Tony Finch <dot@dotat.at>
---
 gitweb/static/gitweb.css | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gitweb/static/gitweb.css b/gitweb/static/gitweb.css
index a869be1..3b4d833 100644
--- a/gitweb/static/gitweb.css
+++ b/gitweb/static/gitweb.css
@@ -68,12 +68,13 @@ div.page_path {
 }
 
 div.page_footer {
-	height: 17px;
+	height: 22px;
 	padding: 4px 8px;
 	background-color: #d9d8d1;
 }
 
 div.page_footer_text {
+	line-height: 22px;
 	float: left;
 	color: #555555;
 	font-style: italic;
-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH v4 3/4] gitweb: omit the repository owner when it is unset
  2013-08-20 17:22         ` [PATCH v4 0/4] Four small gitweb tweaks Tony Finch
  2013-08-20 16:59           ` [PATCH v4 1/4] gitweb: Ensure OPML text fits inside its box Tony Finch
  2013-08-20 16:59           ` [PATCH v4 2/4] gitweb: vertically centre contents of page footer Tony Finch
@ 2013-08-20 16:59           ` Tony Finch
  2013-08-20 16:59           ` [PATCH v4 4/4] gitweb: make search help link less ugly Tony Finch
  3 siblings, 0 replies; 20+ messages in thread
From: Tony Finch @ 2013-08-20 16:59 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski, Jonathan Nieder

On the repository summary page, leave the owner line out if the
repo does not have an owner, rather than displaying a labelled empty
field. This does not affect the owner column in the projects list
page, which is present unless $omit_owner is true.

Signed-off-by: Tony Finch <dot@dotat.at>
---
 gitweb/gitweb.perl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 8d69ada..c029b98 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -6463,7 +6463,7 @@ sub git_summary {
 	print "<div class=\"title\">&nbsp;</div>\n";
 	print "<table class=\"projects_list\">\n" .
 	      "<tr id=\"metadata_desc\"><td>description</td><td>" . esc_html($descr) . "</td></tr>\n";
-        unless ($omit_owner) {
+        if ($owner and not $omit_owner) {
 	        print  "<tr id=\"metadata_owner\"><td>owner</td><td>" . esc_html($owner) . "</td></tr>\n";
         }
 	if (defined $cd{'rfc2822'}) {
-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH v4 4/4] gitweb: make search help link less ugly
  2013-08-20 17:22         ` [PATCH v4 0/4] Four small gitweb tweaks Tony Finch
                             ` (2 preceding siblings ...)
  2013-08-20 16:59           ` [PATCH v4 3/4] gitweb: omit the repository owner when it is unset Tony Finch
@ 2013-08-20 16:59           ` Tony Finch
  3 siblings, 0 replies; 20+ messages in thread
From: Tony Finch @ 2013-08-20 16:59 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski, Jonathan Nieder

The search help link was a superscript question mark right next to
a drop-down menu, which looks misaligned and is a cramped and
awkward click target. Remove the superscript tags and add some
spacing to fix these nits. Add a title attribute to provide an
explanatory mouseover.

Signed-off-by: Tony Finch <dot@dotat.at>
---
 gitweb/gitweb.perl | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 8d69ada..59af7de 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -4030,8 +4030,8 @@ sub print_search_form {
 	      $cgi->input({-name=>"h", -value=>$search_hash, -type=>"hidden"}) . "\n" .
 	      $cgi->popup_menu(-name => 'st', -default => 'commit',
 	                       -values => ['commit', 'grep', 'author', 'committer', 'pickaxe']) .
-	      $cgi->sup($cgi->a({-href => href(action=>"search_help")}, "?")) .
-	      " search:\n",
+	      " " . $cgi->a({-href => href(action=>"search_help"),
+			     -title => "search help" }, "?") . " search:\n",
 	      $cgi->textfield(-name => "s", -value => $searchtext, -override => 1) . "\n" .
 	      "<span title=\"Extended regular expression\">" .
 	      $cgi->checkbox(-name => 'sr', -value => 1, -label => 're',
-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [PATCH v4 0/4] Four small gitweb tweaks
  2013-08-05 11:36       ` Tony Finch
@ 2013-08-20 17:22         ` Tony Finch
  2013-08-20 16:59           ` [PATCH v4 1/4] gitweb: Ensure OPML text fits inside its box Tony Finch
                             ` (3 more replies)
  0 siblings, 4 replies; 20+ messages in thread
From: Tony Finch @ 2013-08-20 17:22 UTC (permalink / raw)
  To: git; +Cc: Jakub Narębski, Jonathan Nieder

This is mostly just a repost to un-stall this topic.

I have fixed the tab damage problem spotted by Jakub in the search
help link patch, and I have improved the commit message for the
repository owner patch. No other changes.

Tony Finch (4):
  gitweb: Ensure OPML text fits inside its box.
  gitweb: vertically centre contents of page footer
  gitweb: omit the repository owner when it is unset
  gitweb: make search help link less ugly

 gitweb/gitweb.perl | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
1.8.3.1.605.g85318f5

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH v4 1/4] gitweb: Ensure OPML text fits inside its box.
  2013-08-20 16:59           ` [PATCH v4 1/4] gitweb: Ensure OPML text fits inside its box Tony Finch
@ 2013-08-20 19:52             ` Junio C Hamano
  0 siblings, 0 replies; 20+ messages in thread
From: Junio C Hamano @ 2013-08-20 19:52 UTC (permalink / raw)
  To: Tony Finch; +Cc: git, Jakub Narębski, Jonathan Nieder

Tony Finch <dot@dotat.at> writes:

> The rss_logo CSS style has a fixed width which is too narrow for
> the string "OPML". Replace the fixed width with horizontal padding
> so the text fits with nice margins.

Makes sense to me (although I do not do css).

>
> Signed-off-by: Tony Finch <dot@dotat.at>
> ---
>  gitweb/static/gitweb.css | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/gitweb/static/gitweb.css b/gitweb/static/gitweb.css
> index cb86d2d..a869be1 100644
> --- a/gitweb/static/gitweb.css
> +++ b/gitweb/static/gitweb.css
> @@ -548,8 +548,7 @@ a.linenr {
>  
>  a.rss_logo {
>  	float: right;
> -	padding: 3px 0px;
> -	width: 35px;
> +	padding: 3px 5px;
>  	line-height: 10px;
>  	border: 1px solid;
>  	border-color: #fcc7a5 #7d3302 #3e1a01 #ff954e;

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH v4 2/4] gitweb: vertically centre contents of page footer
  2013-08-20 16:59           ` [PATCH v4 2/4] gitweb: vertically centre contents of page footer Tony Finch
@ 2013-08-20 19:53             ` Junio C Hamano
  2013-08-21  9:06               ` Tony Finch
  0 siblings, 1 reply; 20+ messages in thread
From: Junio C Hamano @ 2013-08-20 19:53 UTC (permalink / raw)
  To: Tony Finch; +Cc: git, Jakub Narębski, Jonathan Nieder

Tony Finch <dot@dotat.at> writes:

> Signed-off-by: Tony Finch <dot@dotat.at>
> ---
>  gitweb/static/gitweb.css | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/gitweb/static/gitweb.css b/gitweb/static/gitweb.css
> index a869be1..3b4d833 100644
> --- a/gitweb/static/gitweb.css
> +++ b/gitweb/static/gitweb.css
> @@ -68,12 +68,13 @@ div.page_path {
>  }
>  
>  div.page_footer {
> -	height: 17px;
> +	height: 22px;
>  	padding: 4px 8px;
>  	background-color: #d9d8d1;
>  }
>  
>  div.page_footer_text {
> +	line-height: 22px;
>  	float: left;
>  	color: #555555;
>  	font-style: italic;

Hmmm, is it a good idea to do "px" here, or are they ways to do
relative to x-height or something to make sure the text fits?

^ permalink raw reply	[flat|nested] 20+ messages in thread

* Re: [PATCH v4 2/4] gitweb: vertically centre contents of page footer
  2013-08-20 19:53             ` Junio C Hamano
@ 2013-08-21  9:06               ` Tony Finch
  0 siblings, 0 replies; 20+ messages in thread
From: Tony Finch @ 2013-08-21  9:06 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git, Jakub Narębski, Jonathan Nieder, Tony Finch

Junio C Hamano <gitster@pobox.com> wrote:
> Tony Finch <dot@dotat.at> writes:
>
> >  div.page_footer {
> > -	height: 17px;
> > +	height: 22px;
> >  	padding: 4px 8px;
> >  	background-color: #d9d8d1;
> >  }
> >
> >  div.page_footer_text {
> > +	line-height: 22px;
> >  	float: left;
> >  	color: #555555;
> >  	font-style: italic;
>
> Hmmm, is it a good idea to do "px" here, or are they ways to do
> relative to x-height or something to make sure the text fits?

Good question. I also don't know much about css. I basically followed the
style that was already there, and found out about vertical centering using
line-height by searching the web.

I think font-size relative scaling would require a bigger overhaul.

Tony.
-- 
f.anthony.n.finch  <dot@dotat.at>  http://dotat.at/
Forties, Cromarty: East, veering southeast, 4 or 5, occasionally 6 at first.
Rough, becoming slight or moderate. Showers, rain at first. Moderate or good,
occasionally poor at first.

^ permalink raw reply	[flat|nested] 20+ messages in thread

end of thread, other threads:[~2013-08-21  9:06 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-07-09 16:43 [PATCH] gitweb: Ensure OPML text fits inside its box Tony Finch
2013-07-09 17:08 ` [PATCH] gitweb: vertically centre contents of page footer Tony Finch
2013-07-14 23:49 ` [PATCH] gitweb: Ensure OPML text fits inside its box Jonathan Nieder
2013-07-02 16:24   ` [PATCH 3/4] gitweb: omit the repository owner when it is unset Tony Finch
2013-07-09 16:43   ` [PATCH 1/4] gitweb: Ensure OPML text fits inside its box Tony Finch
2013-07-09 17:08   ` [PATCH 2/4] gitweb: vertically centre contents of page footer Tony Finch
2013-07-09 17:15   ` [PATCH 4/4] gitweb: make search help link less ugly Tony Finch
2013-07-15  9:15   ` [PATCH] gitweb: Ensure OPML text fits inside its box Tony Finch
     [not found]   ` <E1Uyeth-0003TF-Kx@hermes-2.csi.cam.ac.uk>
2013-08-03 21:37     ` [PATCH 4/4] gitweb: make search help link less ugly Jakub Narębski
2013-08-05 11:19       ` Tony Finch
     [not found]   ` <E1Uyete-0003Sv-OI@hermes-2.csi.cam.ac.uk>
2013-08-03 21:40     ` [PATCH 3/4] gitweb: omit the repository owner when it is unset Jakub Narębski
2013-08-05 11:36       ` Tony Finch
2013-08-20 17:22         ` [PATCH v4 0/4] Four small gitweb tweaks Tony Finch
2013-08-20 16:59           ` [PATCH v4 1/4] gitweb: Ensure OPML text fits inside its box Tony Finch
2013-08-20 19:52             ` Junio C Hamano
2013-08-20 16:59           ` [PATCH v4 2/4] gitweb: vertically centre contents of page footer Tony Finch
2013-08-20 19:53             ` Junio C Hamano
2013-08-21  9:06               ` Tony Finch
2013-08-20 16:59           ` [PATCH v4 3/4] gitweb: omit the repository owner when it is unset Tony Finch
2013-08-20 16:59           ` [PATCH v4 4/4] gitweb: make search help link less ugly Tony Finch

git@vger.kernel.org list mirror (unofficial, one of many)

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://public-inbox.org/git
	git clone --mirror http://ou63pmih66umazou.onion/git
	git clone --mirror http://czquwvybam4bgbro.onion/git
	git clone --mirror http://hjrcffqmbrq6wope.onion/git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V1 git git/ https://public-inbox.org/git \
		git@vger.kernel.org
	public-inbox-index git

Example config snippet for mirrors.
Newsgroups are available over NNTP:
	nntp://news.public-inbox.org/inbox.comp.version-control.git
	nntp://7fh6tueqddpjyxjmgtdiueylzoqt6pt7hec3pukyptlmohoowvhde4yd.onion/inbox.comp.version-control.git
	nntp://ie5yzdi7fg72h7s4sdcztq5evakq23rdt33mfyfcddc5u3ndnw24ogqd.onion/inbox.comp.version-control.git
	nntp://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/inbox.comp.version-control.git
	nntp://news.gmane.io/gmane.comp.version-control.git
 note: .onion URLs require Tor: https://www.torproject.org/

code repositories for project(s) associated with this inbox:

	https://80x24.org/mirrors/git.git

AGPL code for this site: git clone https://public-inbox.org/public-inbox.git