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

fix(test): removal of select-all shortcut #8165

Merged
merged 3 commits into from Dec 30, 2022
Merged

Conversation

andelf
Copy link
Collaborator

@andelf andelf commented Dec 30, 2022

mod+a is now registered as a global shortcut.

Releated #7803

Bug1:

  • when editing page title, mod+a will ignore current editing status and select page blocks

Bug2:

  • when viewing "Unlinked References", mod+a will cause error:

image

@@ -953,7 +953,7 @@ Similar to re-frame subscriptions"
(set-selection-blocks! blocks :down))
([blocks direction]
(when (seq blocks)
(let [blocks (util/sort-by-height blocks)]
(let [blocks (util/sort-by-height (remove nil? blocks))]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Avoids:

Uncaught TypeError: Cannot read properties of null (reading 'getBoundingClientRect')
    at Object.frontend$util$get_dom_top [as get_dom_top]

@andelf andelf changed the title fix(test): removal of select-all shortcut WIP: fix(test): removal of select-all shortcut Dec 30, 2022
Copy link
Collaborator

@cnrpman cnrpman left a comment

Choose a reason for hiding this comment

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

LGTM

@andelf andelf merged commit fdfac2b into master Dec 30, 2022
@andelf andelf deleted the fix/e2e-select-all-change branch December 30, 2022 08:26
@tiensonqin tiensonqin changed the title WIP: fix(test): removal of select-all shortcut fix(test): removal of select-all shortcut Dec 31, 2022
@Bad3r Bad3r added this to the 0.8.16 milestone Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants