-
Notifications
You must be signed in to change notification settings - Fork 25
<ol> and <ul> menus #146
Comments
I believe it is to provide short-cuts. https://github.com/ucla/mwf/wiki/API%3A-CSS%3A-Menu On 6/21/12 8:08 AM, "Shawn DeArmond"
|
More info at https://github.com/ucla/mwf/wiki/General%3A-Markup-Recommendations
Sounds like somebody believed that |
Rich and Ike are correct that However, I am curious, what is the hesitation in using |
I think the main argument would be that, in many cases, you don't care about the order of the menu items in a navigation menu so a Here are a few people stating the case better than me: Of course, it's a thin line since you do care about the order in that elements towards the top of the list would be more prominent, but there's no logical ordering amongst the set of navigation items themselves. Not sure if the semantic gain outweighs the accessibility loss, though. If it's not difficult to also support |
Not too difficult so we'll add this in a revision release of 1.3. |
This is awesome, folks. Just getting back to it now, but I'm really impressed with the speed in which you all followed up on this issue. I now understand why ol was originally selected, and if ul is added to 1.3, then that would be great. Really, the only reason I asked is that I was working on the Drupal theme and found that Drupal outputs its menus as ul lists by default. It's not terribly difficult to rewrite the theme function so it outputs as ol, but I always try to keep everything as vanilla as possible. Thanks again! |
Completely understandable. The fact that Drupal uses |
If you have access, you can follow my work on the MWF Drupal 7 theme here: It's pretty stable now, except that the menu will look wonky until |
@dearmond I have committed a couple fixes that make |
Is there a reason why the menu items have to be ordered lists? It doesn't display the ordered numbers/letters or anything, so I don't see why it's necessary.
As a solution, I would like to see unordered lists, <ul>, work similarly to ordered lists, <ol>, for menus.
The text was updated successfully, but these errors were encountered: