Header Submenus

There seems to be a bug now when Fixed mode (always show the menu, even when scrolled down) is turned on.

I’m seeing this with Discourse 2.9.0.beta8, and also just reproduced it easily using the official Discourse theme preview site. I think I first noticed it about 2 weeks ago but I’m not entirely sure when it started happening.

To reproduce:

  • Turn on Fixed mode in the theme component’s settings.
  • Click on a thread with a lot of replies.
  • (If it’s the first time you’ve read the thread, and you’re at the very top, scroll down a few replies, go back to the thread list, then click the thread again, so you’re taken back to the post you were on.)

The amount of padding needed at the top of the page is miscalculated, resulting in the top of the post, and the name of the person who wrote it, being scrolled under the header and cut off.

You have to scroll up a bit every time you click on a thread, just to see whose post you’re reading.

You see this on entering the thread:

When you should see something like this:

What’s really strange is it does not always happen, and pushing Ctrl-F5 often fixes it. But it’ll happen again a little while later.

Editing the CSS to increase the amount of padding it adds to main-outlet sort-of fixes things, but then you get a lot of blank space at the top of topic lists. It seems like the topic lists and threads need different padding, or maybe one of them is adding the padding twice or some extra padding from somewhere else?

(Edit: Sometimes the page can’t scroll up far enough to see the problem. For example, when you go to the last post in a thread and it’s a short one, there’s no way for the header to overlap the poster’s name there, even if you scroll down as far as you can manually. That may account for some situations where things appear to work. Test with a browser window that isn’t very tall for the most reliable way to reproduce the bug.)

3 Likes

Is there any way to prevent this from showing on the login screen? I can do it with the following css, but this still presents the header submenu for a second, and then hides it.

.static-login .top-menu {
	display: none;
}
.static-login .d-header-wrap {
	display: none;
}

Is there a better way to prevent this from appearing during the static-login page?

1 Like

This and other themes will not work on the crawler view; the crawler view doesn’t load most of Discourse’s JavaScript.

3 Likes

Hey Kris, this component could use an update for use with the chat and sidebar. With the fixed_mode setting turned on, the chat and sidebar do not get moved down and sit behind the d-header.

The CSS below seems to fix things up nicely, except it doesn’t play well with Category Banners as it seems to be bumping the #main-outlet down an additional 40px on category pages. Thanks!

.sidebar-wrapper, .main-chat-outlet, .chat-drawer {
    margin-top: 40px !important;
}
2 Likes

Thanks for pointing this out! I’ve just added some fixes here:

1 Like

Awesome, thank you Kris! Any idea why the #main-outlet is dropping down 40px?

#main-outlet {
   padding-top: calc(1.8em + 40px);
}

2 Likes

Ah I overlooked that — that can be removed, and will be once this PR is merged: UX: remove old main-outlet adjustment by awesomerobot · Pull Request #19 · discourse/discourse-header-submenus · GitHub

Update: that’s now merged, so the extra space will go away if you update the component.

2 Likes

Where would a guy find the icon names?

3 Likes

I believe everything listed here is available for use: Find the Perfect Icon for Your Project in Font Awesome 5 | Font Awesome

Though some icons require an additional step… for icons we don’t use in Discourse by default the icon names needs to be added to the Svg icons theme setting.

If you really want to be specific, all the icons we use in Discourse by default are listed here: discourse/svg_sprite.rb at da0d20d4a99aab40bd08bf133a425be413521d5a · discourse/discourse · GitHub

And all the icons available for use in themes (via the svg icons setting) are in this directory: discourse/vendor/assets/svg-icons at main · discourse/discourse · GitHub

Viewing the SVG files in github isn’t really helpful, because you’ll see a big white box:

but you can click the raw button and view the page source, the icon name is the symbol’s id

3 Likes

You rock, thanks so much

2 Likes

Thanks Kris, that fixed it. I have noticed a couple of other issues though. .topic-navigation needs to be pushed down 40px, and .topic-post.sticky-avatar .topic-avatar seems to be calculated incorrectly when you scroll down, and it ends up behind the d-header. Also, it’s possible to expand the composer and chat window too high and lose the controls behind the d-header as well. I think that should cover everything!

Edit: one more thing, when you visit an anchor link the heading is under the header as well.

2 Likes

2 posts were split to a new topic: The component pushes down the chat, hiding partially the chat input

2 posts were merged into an existing topic: The component pushes down the chat, hiding partially the chat input

Did anyone figure this out? I like the approach of not using additional space and reusing the resources already on the screen.

It’s really cool that hovering over the logo allows to navigate to blog, docs, or wiki!

2 Likes

FWIW the “svg icons theme setting” mentioned by Kris can currently be found in Admin > Settings > Other > towards the bottom there’s svg icon subset (…I think this is correct??)

1 Like

There is a small z-index issue with the new Glimmer lightbox, 1001 just puts it above the lightbox because the lightbox is at 1000.

2 Likes

Classic z-index problem, it will be fixed the next time Discourse is updated

4 Likes

Thanks for this nice plugin.

I wonder, is there a way not to display the header submenus when not logged in ?

Or, (as idea of ​​development ?) may be better, what about having menu items displayed link to groups (trust levels or others) ?
Easy to ask for i know…

Thanks
C.

2 Likes

I don’t think this is anywhere else, apologies if so. I love using header submenus, but I have a lot of top headers, and it looks quite poor on mobile as a result. I would like to have it so that my headers don’t extend off-screen and cause me to scroll. Is this possible?

2 Likes

Okay I sorta solved my issue by just compiling all of the most important links as sub-menus into a “Navigational” top header on mobile.

2 Likes