diff --git a/menu/skeleton-parts/player.js b/menu/skeleton-parts/player.js index b3880cef1..e5ccc0e09 100644 --- a/menu/skeleton-parts/player.js +++ b/menu/skeleton-parts/player.js @@ -1814,7 +1814,7 @@ extension.skeleton.main.layers.section.player.on.click = { component: "button", text: "hidePlayerControlsBarButtons", on: { - click: 'main.layers.section.appearance.on.click.player.on.click.player_hide_controls_options.on.click' + click: 'main.layers.section.appearance.on.click.player.on.click.section_1.player_hide_controls_options.on.click' } } } diff --git a/tests/unit/player-controls-menu.test.js b/tests/unit/player-controls-menu.test.js new file mode 100644 index 000000000..dc6cfd38b --- /dev/null +++ b/tests/unit/player-controls-menu.test.js @@ -0,0 +1,35 @@ +global.extension = { + skeleton: { + main: { + layers: { + section: {} + } + } + } +}; + +global.satus = { + storage: { + onchanged: jest.fn() + } +}; + +require('../../menu/skeleton-parts/appearance.js'); +require('../../menu/skeleton-parts/player.js'); + +describe('Player controls menu redirect (#4215)', () => { + test('opens the shared hide-controls options section', () => { + const playerSection = extension.skeleton.main.layers.section.player.on.click; + const redirect = playerSection.section_2.player_hide_controls_options.on.click; + const target = redirect + .split('.') + .reduce((skeleton, key) => skeleton[key], extension.skeleton); + + expect(target).toBe( + extension.skeleton.main.layers.section.appearance.on.click.player.on.click + .section_1.player_hide_controls_options.on.click + ); + expect(target.component).toBe('section'); + expect(target.player_play_button.component).toBe('switch'); + }); +});