Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1487)

Issue 193140043: Web: revise navigation bars with CSS gradients (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 3 months ago by pwm
Modified:
9 years, 3 months ago
Reviewers:
pkx166h
CC:
lilypond-devel_gnu.org
Base URL:
http://git.savannah.gnu.org/gitweb/?p=lilypond.git/trunk/
Visibility:
Public.

Description

Web: revise navigation bars with CSS gradients Includes background image fallbacks for IE9 and lower in separate, conditional CSS file.

Patch Set 1 #

Patch Set 2 : fix a typo in lilypond-website.css #

Patch Set 3 : add images #

Patch Set 4 : same patch, rebased on current master #

Unified diffs Side-by-side diffs Delta from patch set Stats (+85 lines, -137 lines) Patch
M Documentation/css/lilypond-website.css View 12 chunks +48 lines, -119 lines 0 comments Download
M Documentation/css/lilypond-website-ie-fixes.css View 2 chunks +35 lines, -0 lines 0 comments Download
M Documentation/lilypond-texi2html.init View 2 chunks +2 lines, -18 lines 0 comments Download
A Documentation/pictures/lily-home-32x32.png View Binary file 0 comments Download
A Documentation/pictures/nav1-active.png View Binary file 0 comments Download
A Documentation/pictures/nav1-bg.png View Binary file 0 comments Download
A Documentation/pictures/nav1-hover.png View Binary file 0 comments Download
A Documentation/pictures/nav2-active.png View Binary file 0 comments Download
A Documentation/pictures/nav2-bg.png View Binary file 0 comments Download
A Documentation/pictures/nav2-hover.png View Binary file 0 comments Download

Messages

Total messages: 8
pwm
Please review. A demo showing the desired result can be found here: http://clairnote.org / lilypond-web-demo2/ ...
9 years, 3 months ago (2015-01-14 18:12:54 UTC) #1
pwm
The images are now included in this patch for lilypond-git and also in a pull-request ...
9 years, 3 months ago (2015-01-15 04:32:19 UTC) #2
pkx166h
On 2015/01/15 04:32:19, pwm wrote: > The images are now included in this patch for ...
9 years, 3 months ago (2015-01-16 09:25:19 UTC) #3
pwm
On 2015/01/16 09:25:19, J_lowe wrote: > > Fails to apply to current master. > > ...
9 years, 3 months ago (2015-01-16 15:37:09 UTC) #4
pkx166h
On 2015/01/16 15:37:09, pwm wrote: > On 2015/01/16 09:25:19, J_lowe wrote: > > > > ...
9 years, 3 months ago (2015-01-16 18:26:07 UTC) #5
pwm
On 2015/01/16 18:26:07, J_lowe wrote: > > You forgot to set the tracker to patch-new ...
9 years, 3 months ago (2015-01-17 16:59:29 UTC) #6
pkx166h
On 2015/01/17 16:59:29, pwm wrote: > On 2015/01/16 18:26:07, J_lowe wrote: > > > > ...
9 years, 3 months ago (2015-01-19 18:20:49 UTC) #7
pkx166h
9 years, 3 months ago (2015-01-23 03:49:55 UTC) #8
author	Paul Morris <paulwmorris@gmail.com>	
	Thu, 22 Jan 2015 18:09:39 +0000 (18:09 +0000)
committer	James Lowe <pkx166h@gmail.com>	
	Fri, 23 Jan 2015 03:42:30 +0000 (03:42 +0000)
commit	0a89d9ec5ab4b03a6396c0970aa0a227f8950c31
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b