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

JT-83359 Enable back controlling shortcuts #7637

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

andrey-skl
Copy link
Collaborator

by having '.ring-js-shortcuts' on parent, removed in 473d554

@andrey-skl andrey-skl requested review from vkrol and Hypnosphi August 14, 2024 08:12
@andrey-skl
Copy link
Collaborator Author

@Hypnosphi Don't you remember if there was specific need to call element.closest(this.ALLOW_SHORTCUTS_SELECTOR) only if element has no `data-enabled-shortcuts attribute in this commit 473d554 ?

@andrey-skl andrey-skl merged commit 708b012 into release-6.x Aug 15, 2024
4 checks passed
@andrey-skl andrey-skl deleted the release-6.x-fix-shortcuts branch August 15, 2024 08:47
andrey-skl added a commit that referenced this pull request Aug 19, 2024
andrey-skl added a commit that referenced this pull request Aug 21, 2024
* RG-2293 Get rid of side effects in setState updater function of Select component (#7574)

* 6.0.50

* RG-2462 switch sidebar and main background in dark theme (#7565)

(cherry picked from commit 647dd76)

* 6.0.51

* Support passing theme down via React Context (#7592)

* 6.0.52

* RG-2420 chore: undeprecate "Badge" in 6.x version

* Chore: export GLOBAL_DARK_CLASS_NAME constant [publish]

* 6.0.53

* JT-83359 Enable back controlling shortcuts by having '.ring-js-shortcuts' on parent, removed in 473d554 (#7637)

* RG-2473 don't unmount react root because it can be reused later (#7656)

minor: log auth error

* Revert "RG-2420 chore: undeprecate "Badge" in 6.x version"

This reverts commit 580369f.

---------

Co-authored-by: Veniamin Krol <153412+vkrol@users.noreply.github.com>
Co-authored-by: JetBrains Ring UI Automation <npmjs-buildserver@jetbrains.com>
andrey-skl added a commit that referenced this pull request Sep 4, 2024
* RG-2293 Get rid of side effects in setState updater function of Select component (#7574)

* 6.0.50

* RG-2462 switch sidebar and main background in dark theme (#7565)

(cherry picked from commit 647dd76)

* 6.0.51

* Support passing theme down via React Context (#7592)

* 6.0.52

* RG-2420 chore: undeprecate "Badge" in 6.x version

* Chore: export GLOBAL_DARK_CLASS_NAME constant [publish]

* 6.0.53

* JT-83359 Enable back controlling shortcuts by having '.ring-js-shortcuts' on parent, removed in 473d554 (#7637)

* RG-2473 don't unmount react root because it can be reused later (#7656)

minor: log auth error

* 6.0.54

* Revert "JT-83359 Enable back controlling shortcuts by having '.ring-js-shortcuts' on parent, removed in 473d554"

This reverts commit 9192646.

* 6.0.55

* Put page reload in timeout for USER_CHANGED event to let all other events to be called. JT-83975 related

* 6.0.56

* Chore: add example with dialog with close button inside

* Revert "RG-2420 chore: undeprecate "Badge" in 6.x version"

This reverts commit 580369f.

---------

Co-authored-by: Veniamin Krol <153412+vkrol@users.noreply.github.com>
Co-authored-by: JetBrains Ring UI Automation <npmjs-buildserver@jetbrains.com>
andrey-skl added a commit that referenced this pull request Oct 21, 2024
* RG-2293 Get rid of side effects in setState updater function of Select component (#7574)

* 6.0.50

* RG-2462 switch sidebar and main background in dark theme (#7565)

(cherry picked from commit 647dd76)

* 6.0.51

* Support passing theme down via React Context (#7592)

* 6.0.52

* RG-2420 chore: undeprecate "Badge" in 6.x version

* Chore: export GLOBAL_DARK_CLASS_NAME constant [publish]

* 6.0.53

* JT-83359 Enable back controlling shortcuts by having '.ring-js-shortcuts' on parent, removed in 473d554 (#7637)

* RG-2473 don't unmount react root because it can be reused later (#7656)

minor: log auth error

* 6.0.54

* Revert "JT-83359 Enable back controlling shortcuts by having '.ring-js-shortcuts' on parent, removed in 473d554"

This reverts commit 9192646.

* 6.0.55

* Put page reload in timeout for USER_CHANGED event to let all other events to be called. JT-83975 related

* 6.0.56

* Chore: add example with dialog with close button inside

* RG-2227 Clicking on a select label reopens the dropdown

(cherry picked from commit a685a07)

* chore: Prevent dialog scroll (#7710)

* chore: Prevent dialog scroll

* chore: preventBodyScroll should not be passed as a part of restProps

* 6.0.57

* Add attributes prop to DatePicker button (#7727)

Co-authored-by: Mikhail Cherviakov <mikhail.cherviakov@jetbrains.com>

* 6.0.58

* [publish] fix breadcrumbs work with React.Fragment containers as a transparent containers

reactjs/rfcs#61

* 6.0.59

* Revert "[publish] fix breadcrumbs work with React.Fragment containers as a transparent containers"

This reverts commit 704bb21.

* 6.0.60

* Fix Table colspan calculation when columns is passed as function

* 6.0.61

* Get rid of unused Table maxColSpan prop

* RG-2490 support moving cursor from anchor to tooltip (#7754)

* RG-2490 support moving cursor from anchor to tooltip

* RG-2490 use React for "mouseout" event

* 6.0.62

* Check className in QueryAssist shouldComponentUpdate

* 6.0.63

---------

Co-authored-by: Veniamin Krol <153412+vkrol@users.noreply.github.com>
Co-authored-by: JetBrains Ring UI Automation <npmjs-buildserver@jetbrains.com>
Co-authored-by: Filipp Riabchun <filipp.riabchun@jetbrains.com>
Co-authored-by: Alexander Burkov <a.p.burkov@gmail.com>
Co-authored-by: Mihail Ch. <pltnm239@gmail.com>
Co-authored-by: Mikhail Cherviakov <mikhail.cherviakov@jetbrains.com>
Co-authored-by: maksim.erekhinskii <maxim.erehinskiy@jetbrains.com>
Co-authored-by: Veniamin Krol <veniamin.krol@jetbrains.com>
vkrol added a commit that referenced this pull request Nov 6, 2024
…s-shortcuts' on parent, removed in 473d554 (#7637)"

This reverts commit 708b012.
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.

2 participants