Font for navigation is preloaded in header even when navigation bar is not loaded

raising to major since it interferes with the normal use of the site.

We should be looking at potentially splitting the font up to reduce size see https://github.com/zachleat/glyphhanger

We should be looking at potentially splitting the font up to reduce size

💯💯💯

Good catch 💯

Added logic to include the font if only the toolbar is being rendered.

About font split. Do you think it has to be addressed here? Or might be moved to a lower priority follow up?

RE
#3
#3

We need to check it first:
https://github.com/zachleat/glyphhanger/issues/22

To be sure that variable fonts supported. So i guess better to do it in separated ticket.

Hi,
I've tried to reproduce the issue on Drupal version 11.x
Please let me know if I've missed anything while reproducing the issue

Testing steps:

1. Install Drupal
2. Install Navigation and Navigation Top bar Module
3. and checked with an Anonymous user

Adding screenshot

Just checked in last tugboat. Seems all fine

@Kanchan Bhogade probably cache?

About font split. Do you think it has to be addressed here? Or might be moved to a lower priority follow up?

Definitely a followup. Probably needs a bit of discussion and maybe even testing.

Thanks for working on this! Looks to be working as expected now. In addition to the administrator/anonymous screenshots below, I also created an authenticated user that doesn't have access to use the navigation bar and made sure the font wasn't loading for them.

Looks great!

Opened followup: 📌 Navigation: Discuss/implement splitting variable font Active

+1 on just moving the #attached['html_head_link'] this way, makes loads of sense.

Committed/pushed to 11.x, thanks! Moving to 10.3.x for backport with the rest of the navigation commits.

Automatically closed - issue fixed for 2 weeks with no activity.