Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tabbar: support icon-less items #12804

Merged
merged 2 commits into from
Aug 31, 2023

Commits on Aug 28, 2023

  1. tabbar: fix command contributions regression

    The merge of eclipse-theia#12799 introduced a regression in
    the rendering of MenuToolbarItems that aren't
    pop-up menus but instead just commands that
    happen to have a menu path: they were rendered
    as pop-up menus instead of simple buttons.
    
    This fix refines the condition for the new pop-up
    menu rendering to check that the menu to be
    rendered is not a command to be executed
    directly on click.
    
    Fixes eclipse-theia#12687
    
    Signed-off-by: Christian W. Damus <[email protected]>
    cdamus committed Aug 28, 2023
    Configuration menu
    Copy the full SHA
    b702ee0 View commit details
    Browse the repository at this point in the history
  2. tabbar: support icon-less items

    VS Code renders action buttons in the tab bar
    for commands that do not have icons using
    their title text, instead. This commit does the
    same in Theia.
    
    Additionally, two related minor issues are fixed:
    
    - the $(icon) specifiers for icons show in the
      tooltip of an action, which is confusing
    - the roll-over highlight shows only on action
      buttons for commands that use the "icon"
      property, not those that embed icon specifiers
      in their titles
    
    Fixes eclipse-theia#12686
    
    Signed-off-by: Christian W. Damus <[email protected]>
    cdamus committed Aug 28, 2023
    Configuration menu
    Copy the full SHA
    a608abe View commit details
    Browse the repository at this point in the history