Re: [PHP-WEBMASTER] svn: /web/php/trunk/ images/shade.png include/header.inc include/layout.inc js/common.js js/jquery.hoverIntent.minified.js styles/structure.css styles/theme.css
[email protected] (Hannes Magnusson)
| Newsgroups | php.webmaster |
|---|---|
| Message-ID | <[email protected]> |
On Sat, Jan 1, 2011 at 21:57, Stewart Lord <[email protected]> wrote: > > On 2011-01-01, at 10:56 AM, Hannes Magnusson wrote: > > On Sat, Jan 1, 2011 at 19:30, Stewart Lord <[email protected]> wrote: > > Another, less important issue. Previously the sub-menus lived in the list > with their parent links which from a semantic html perspective is more > correct. By moving the sub-menus around in the dom, we are compromising the > structure of our html to make our JS easier. Can't say I care too much about > that though. > > > One of the reasons why I didn't commit it :] > > I still haven't thought of a better way to fix this easily. > > -Hannes > > > It does make the code more complicated having to move the sub-menus around > in the dom. Does this fix the issue where clicking a menu would steal the > current menu (e.g. if I go to downloads and click doc, downloads is no > longer 'current')? I thought that might require another class to track > state. It.. kinda, yes, I guess. It doesn't set it on the menu, so the "selected menu" doesn't have any different color then usual. Its no problem adding a "selected" class, but I couldn't think of any proper color to use so I dropped it :P > Let me know if you want me to look at this, otherwise I will continue > playing with designs for the home page. Yes/No is all I expect really :) Your time is best spent on the frontpage imo. Once thats done you can worry about the things I may have made ugly :D -Hannes