Top Menu Layout Request

NOYB

When less than 1200 px width top nav horizontal menu is placed below the logo. This consumes too much vertical space, obscuring the page headers. The main cause is the logo space consumption not leaving enough horizontal space for the menu. So it forms a new line below the logo.

jdillard

@NOYB:

When less than 1200 px width top nav horizontal menu is placed below the logo. This consumes too much vertical space, obscuring the page headers. The main cause is the logo space consumption not leaving enough horizontal space for the menu. So it forms a new line below the logo.

I just committed, sorry for not clarifying. did you gitsync or just upgrade?

Did you clear you cache just to be sure?

Also if the issue persists, what browser?

NOYB

I gitsync'd but must not have refreshed. Thanks for the reminder. I like this much better.

Though for menu transition to be in unison with the dashboard transition from single to double column I think the @media (max-width: 992px) should be 991px. With the 992px setting the menu transition is one pixel wider then the dash board columns transition. Kind of creates a flicker effect.

This works well for me.


 @media (max-width: 991px) {
	/* change top navbar from horizontal to vertical */

jdillard

Ok cool, just pushed that change.

Also if you want to generate a notice you can go to Diagnostics > Command Prompt and enter file_notice("test", "test"); in the PHP commands box, then refresh.

I also made the fixed nav a BETA option for now as it was causing issues with the long dropdowns, so you might notice a change there.

edit: TODOs include:

cleaning up notice modal
making the notice badge stay visible in the nav on smaller screens
trying to fit a logout icon up there somehow.

NOYB

Notifications works.

Logout. People actually do that? How long are inactive sessions retained without any keep-alives, etc.?

jdillard

Cool. thanks for the help.

Yea, if anything for peace of mind. I played around with it a little and #2 and #3 on that list and they are going to be low priority to give time to make sure it is implemented correctly.

Now to clean up the modal…

NOYB

For the top nav horizontal menu to expand in unison with widget width the "@media (max-width: 1200px)" should be 1199px. At 1200px the menu and widget widths expand one pixel apart from each other.


@media (max-width: 1199px) {
    .nav>li>a {
        padding: 14px 9px 10px 9px;
    }
 }

JeGr

Logout. People actually do that? How long are inactive sessions retained without any keep-alives, etc.?

Yes people do and should do that. It's been more than one customer where I got access to a browser, that was logged into the system a while ago and I could open and access their pfSense without authenticating. Also users tend to go the easiest route so some of them even disable auto-logout alltogether. When working in larger setups and scenarios, you often have systems with multiple users/administrator levels, so re-logging is something you absolutely do!

@NOYB Perhaps the menu or widget width should be enlarged by 1px instead of reducing even boundaries like (1200)?

jdillard

@NOYB:

For the top nav horizontal menu to expand in unison with widget width the "@media (max-width: 1200px)" should be 1199px. At 1200px the menu and widget widths expand one pixel apart from each other.

Thanks! Just pushed this change.

NOYB

@JeGr:

@NOYB Perhaps the menu or widget width should be enlarged by 1px instead of reducing even boundaries like (1200)?

max 1199 sets the boundary at 1200. So transition happens at 1200 instead of at 1201.