Navbar migration [#2072269]

From http://bootply.com/bootstrap-3-migration-guide

remove .navbar-inner

replace .brand with .navbar-brand

wrap .navbar-brand and .navbar-toggle inside .navbar-header

add .navbar-nav to .nav

From http://blog.getbootstrap.com/2013/08/19/bootstrap-3-released/

Navbars now require a .navbar-default for the standard version.

And from bootstrap new documentation http://getbootstrap.com/components/#navbar

Comment	File	Size	Author
#4	bootstrap-navbar-migration-2072269-4.patch	3.5 KB	el7cosmos
#4
#1	bootstrap-navbar-migration-2072269-1.patch	3.66 KB	el7cosmos
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

el7cosmos

Indonesian

🇮🇩 GMT+7

CreditAttribution: el7cosmos commented 23 August 2013 at 02:31

File	Size
bootstrap-navbar-migration-2072269-1.patch	3.66 KB

Comment #2

el7cosmos

Indonesian

🇮🇩 GMT+7

CreditAttribution: el7cosmos commented 23 August 2013 at 02:32

Status:

Active

» Needs review

Comment #3

markhalliwell

English

CreditAttribution: markhalliwell commented 25 August 2013 at 17:47

Status:

Needs review

» Needs work

+++ b/templates/page.tpl.php
@@ -1,12 +1,13 @@
-      <!-- .btn-navbar is used as the toggle for collapsed navbar content -->
...
+    <!-- Brand and toggle get grouped for better mobile display -->

@@ -15,27 +16,28 @@
+      <!-- Collect the nav links, forms, and other content for toggling -->
...
+      </div><!-- /.navbar-collapse -->

I know we have a few HTML comments, but I'd rather be consistent and just take these out all together.