fix submenu focus

[SpaNb4]

Fixes the following issues:

1. The focus does not shift back when reopening the menu when a submenu item is active

2023-08-09.14-51-25.mp4

2. (preact specific) When a sub-menu opens, I can't use the arrow keys to navigate through it

2023-08-09.14-56-16.mp4

Note: I use the antd Dropdown component in the videos above

[yoyo837]

Please add some test cases to cover this.

[SpaNb4]

Please add some test cases to cover this.

I wanted to add some tests, but then I realized that rc-menu doesn't have an autoFocus prop that antd uses. So I'm not sure if it's possible to reproduce this behavior.
And speaking of the second issue, I don't think it's possible to test it, since you need to use preact in your tests

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d21b6bb) 99.58% compared to head (2ff8bad) 99.58%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #662   +/-   ##
=======================================
  Coverage   99.58%   99.58%           
=======================================
  Files          27       27           
  Lines         721      722    +1     
  Branches      196      196           
=======================================
+ Hits          718      719    +1     
  Misses          3        3           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[zombieJ]

cc @MadCcc

[MadCcc]

Is this issue related?
ant-design/ant-design#43267

[MadCcc]

Change on this file should be reverted since nothing except order changed.

[MadCcc]

I wanted to add some tests, but then I realized that rc-menu doesn't have an autoFocus prop that antd uses. So I'm not sure if it's possible to reproduce this behavior.

Test for the first problem should be provided, you can trigger focus manually in test case to mock the scene.

[SpaNb4]

Is this issue related? ant-design/ant-design#43267

No, this PR is not relevant to the above issue

[SpaNb4]

I wanted to add some tests, but then I realized that rc-menu doesn't have an autoFocus prop that antd uses. So I'm not sure if it's possible to reproduce this behavior.

Test for the first problem should be provided, you can trigger focus manually in test case to mock the scene.

Ok, I'll try

[yoyo837]

Now it conflicts.

[SpaNb4]

Closing this since I can no longer reproduce the first issue