twitter/bootstrap — Issue #1216: 2.0-wip Improve HTML for nav list with icons
Author: kylewest
Created: 2012-01-25 19:57:10+00:00
State: open

**Note: I will submit a pull request for this, before I do the work though I wanted to make sure there isn't some reason for the current implementation.**

Nav lists with icons are defined like so:

```html

List header

Home

Library

Applications

Another list header

Profile

Settings

Help

```

Why are we using `` within the `` tag? This:

```html
Library

```

could be rewritten to this:

```html
Library

```

or this:

```html
Library

```

without affecting the end result.

Should I do the work and submit a PR, or is this coded as-is for a specific reason? What is the preferred markup (I like the 1st option)? Should the change maintain backwards compatibility?

Thanks.

Kyle