Merge lp:~cjohnston/ubuntu-community-webthemes/top-nav into lp:ubuntu-community-webthemes/light-django-theme

Proposed by Chris Johnston
Status: Merged
Merged at revision: 32
Proposed branch: lp:~cjohnston/ubuntu-community-webthemes/top-nav
Merge into: lp:ubuntu-community-webthemes/light-django-theme
Diff against target: 56 lines (+8/-10)
2 files modified
media/css/default.css (+4/-4)
templates/ubuntu_website_base.html (+4/-6)
To merge this branch: bzr merge lp:~cjohnston/ubuntu-community-webthemes/top-nav
Reviewer Review Type Date Requested Status
Ubuntu Community Web Themes Developers Pending
Review via email: mp+52790@code.launchpad.net
To post a comment you must log in.
33. By Chris Johnston

fixes background color of top-nav

34. By Chris Johnston

Made fixes suggested by Ronnie

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'css_search.pl' (properties changed: +x to -x)
2=== modified file 'media/css/default.css'
3--- media/css/default.css 2011-02-26 15:06:54 +0000
4+++ media/css/default.css 2011-03-14 19:16:06 +0000
5@@ -233,7 +233,7 @@
6
7 .top-nav-item {
8 display:inline-block;
9- padding:0 8px;
10+ margin-right: 14px;
11 line-height:2em;
12 }
13
14@@ -250,7 +250,7 @@
15
16 #top-related {
17 float:right;
18- margin-right:15px;
19+ margin-right:2px;
20 }
21
22 #main-nav {
23@@ -596,10 +596,10 @@
24 }
25
26 #top-nav {
27- background-color:#f7f7f7;
28+ background-color:#ffffff;
29 }
30
31-.top-nav-item {
32+.top-nav-item, .top-nav-item:visited {
33 color:#333333;
34 }
35
36
37=== modified file 'templates/ubuntu_website_base.html'
38--- templates/ubuntu_website_base.html 2011-02-26 15:06:54 +0000
39+++ templates/ubuntu_website_base.html 2011-03-14 19:16:06 +0000
40@@ -49,12 +49,10 @@
41 </section>
42 <nav id="top-related">
43 {% block top_nav_related_links %}
44- <a class="top-nav-item" href="www.ubuntu.com">Ubuntu.com</a>
45- <a class="top-nav-item" href="ubuntuforums.org">Support</a>
46- <a class="top-nav-item" href="wiki.ubuntu.com">Documentation</a>
47- <a class="top-nav-item" href="fridge.ubuntu.com">News</a>
48- <a class="top-nav-item" href="planet.ubuntu.com">Blogs</a>
49- <a class="top-nav-item" href="loco.ubuntu.com">Loco Directory</a>
50+ <a class="top-nav-item" href="http://www.ubuntu.com">Ubuntu.com</a>
51+ <a class="top-nav-item" href="http://www.ubuntu.com/community">Community</a>
52+ <a class="top-nav-item" href="http://www.ubuntu.com/support">Support</a>
53+ <a class="top-nav-item" href="http://www.ubuntu.com/partners">Partners</a>
54 {% endblock %}
55 </nav>
56 </div>

Subscribers

People subscribed via source and target branches