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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.