Skip to content

Learning Header's user menu slots can't fully hide/replace it. #675

Closed

Description

@tecoholic

Description

The learning MFE's user menu is made up of 2 slots

  • org.openedx.frontend.layout.header_learning_user_menu_toggle.v1
  • org.openedx.frontend.layout.header_learning_user_menu.v1
    :
    They wrap the toggle button the menu's contents, but not the dropdown itself. So the dropdown cannot be fully hidden even when both the slots are set to hide.

Example config snippet:

  'org.openedx.frontend.layout.header_learning_user_menu_toggle.v1': {
    plugins: [
      {
        widgetId: 'default_contents',
        op: PLUGIN_OPERATIONS.Hide,
      },
    ],
  },
  'org.openedx.frontend.layout.header_learning_user_menu.v1': {
    plugins: [
      {
        widgetId: 'default_contents',
        op: PLUGIN_OPERATIONS.Hide,
      }
    ]
  },

Artifact with the config applied

Note the dropdown and empty menu on the right corner

Image

Potential Workaround and second bug

The Learning MFE provides a slot that covers the full Header - org.openedx.frontend.layout.header_learning.v1, that has the param showUserDropdown. This in theory should offer an workaround with a configuration like

const hideUser = (widget) => {
  console.log("Hide User:", widget)
  widget.content = {
    ...widget.content,
    showUserDropdown: false,
  };
  return widget;
}

const config = {
  pluginSlots: {
    'org.openedx.frontend.layout.header_learning.v1': {
      keepDefault: true,
      plugins: [
      {
        op: PLUGIN_OPERATIONS.Modify,
        widgetId: 'default_contents',
        fn: hideUser,
      }
    ]
    },
  }
};

And then any potential replacement could be added to the LearningHeaderActionSlot.

However, the LearningHeaderActionSlot is wrapped in the same block as the User dropdown, making the workaround infeasible.

Activity

  1. added
    bugReport of or fix for something that isn't working as intended
    on Jul 24, 2026
  2. added theissue type on Aug 7, 2026
  3. removed
    bugReport of or fix for something that isn't working as intended
    on Aug 7, 2026
  4. arbrandes commented on Aug 7, 2026

    @arbrandes
    Contributor

    Interesting. Is this also reproducible in frontend-base land? If it is, I'd like us to fix it there first.

  5. added this to the Willow Cutoff milestone on Aug 7, 2026
  6. tecoholic commented on Aug 10, 2026

    @tecoholic
    ContributorAuthor

    @arbrandes I haven't tested it with the new frontend-base. I will investigate it this week.

  7. linked a pull request that will close this issuefeat: Add V2 of header actions slot #682on Aug 27, 2026
  8. moved this from Todo to In Progress in Frontend Working Groupon Aug 27, 2026
  9. moved this from In Progress to In Review in Frontend Working Groupon Aug 28, 2026
  10. arbrandes commented on Sep 2, 2026

    @arbrandes
    Contributor

    Answering my own question from August: no, this is not reproducible in frontend-base, so it doesn't block a fix here.

  11. tecoholic commented on Sep 3, 2026

    @tecoholic
    ContributorAuthor

    @arbrandes Thanks. cc: @ChrisChV

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions