Skip to content

fix: improve global contextMenu in activity bar#200

Merged
wewoor merged 3 commits intomainfrom
fix/activityBar
Jun 24, 2021
Merged

fix: improve global contextMenu in activity bar#200
wewoor merged 3 commits intomainfrom
fix/activityBar

Conversation

@mortalYoung
Copy link
Copy Markdown
Collaborator

简介

  • 优化 activityBar 中 global item 点击唤起菜单的方式和最顶部的 menuBar 不一样的问题。menuBar 是通过 DropDown 组件来唤醒,会去计算位置,由于点击唤起菜单和右键点击唤起菜单不同,所以这里选择也用 DropDown 的方式

主要变更

  • 优化 activityBarItem 中的点击事件
  • 优化 DropDown 中的 placement 的类型,新增 rightBottom 表示支持居右并底部对齐

Comment thread src/common/dom.ts Outdated
@wewoor wewoor merged commit 7ad58ab into main Jun 24, 2021
@wewoor wewoor deleted the fix/activityBar branch June 24, 2021 09:30
Comment thread src/common/dom.ts
y = domRect.y + domRect.height;
} else if (placement === 'left') {
x = domRect.x - domRect.width;
} else if (placement === 'rightBottom') {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
} else if (placement === 'rightBottom') {
} else if (placement === 'bottomRight') {

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants