It looks like the login link was moved into the <ul>, which looks good. I like it.
But if you login, the user box also then becomes a <li>, which pushes the list up. Furthermore, it then makes the logout button into a sub-menu of the user box, and logout cannot be selected.
I think the most important thing is that the logout button is clickable. If that's fixed, everything else can be ignored until it becomes the highest priority item left.
jhauserw3241
commented
7 years ago
Let me define wonky.
It looks like the login link was moved into the
<ul>, which looks good. I like it.But if you login, the user box also then becomes a
<li>, which pushes the list up. Furthermore, it then makes the logout button into a sub-menu of the user box, and logout cannot be selected.I think the most important thing is that the logout button is clickable. If that's fixed, everything else can be ignored until it becomes the highest priority item left.