Skip to content

Commit

Permalink
[core] fix(Hotkeys): Fix hotkeys dialog having two scrollbars (#6762)
Browse files Browse the repository at this point in the history
Co-authored-by: Nikita Morozov <nmorozov@palantir.com>
  • Loading branch information
okaynik and Nikita Morozov committed Apr 19, 2024
1 parent 2fa0f5d commit a19169c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
2 changes: 0 additions & 2 deletions packages/core/src/components/hotkeys/_hotkeys.scss
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,6 @@

.#{$ns}-hotkey-column {
margin: auto;
max-height: 80vh;
overflow-y: auto;
padding: $pt-grid-size * 3;

.#{$ns}-heading {
Expand Down
4 changes: 2 additions & 2 deletions packages/core/src/legacy/hotkeysDialogLegacy.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ import * as React from "react";
import * as ReactDOM from "react-dom";

import { Classes } from "../common";
import { Dialog, type DialogProps, Hotkey, type HotkeyProps, Hotkeys } from "../components";
import { Dialog, DialogBody, type DialogProps, Hotkey, type HotkeyProps, Hotkeys } from "../components";

interface HotkeysDialogProps extends DialogProps {
/**
Expand Down Expand Up @@ -125,7 +125,7 @@ class HotkeysDialogLegacy {
isOpen={this.isDialogShowing}
onClose={this.hide}
>
<div className={Classes.DIALOG_BODY}>{this.renderHotkeys()}</div>
<DialogBody>{this.renderHotkeys()}</DialogBody>
</Dialog>
);
}
Expand Down

1 comment on commit a19169c

@svc-palantir-github
Copy link

Choose a reason for hiding this comment

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

[core] fix(`Hotkeys`): Fix hotkeys dialog having two scrollbars (#6762)

Build artifact links for this commit: documentation | landing | table | demo

This is an automated comment from the deploy-preview CircleCI job.

Please sign in to comment.