Don't automatically redirect to the "New Function" page when pressing the "Enter" key on the "/settings/functions" page

[Devessier]

Bug Description

Wherever the focus is on the "/settings/functions" page, pressing the "Enter" key will always navigate to the "New Function" page. This breaks accessibility.

Example:

CleanShot.2024-09-24.at.12.24.03.mp4

Expected behavior

The page should not listen to the Enter key and let the browser behave naturally.

Technical inputs

Remove that code:

twenty/packages/twenty-front/src/modules/settings/serverless-functions/components/SettingsServerlessFunctionsTable.tsx

Lines 34 to 40 in fcaa9d9

	useScopedHotkeys(
	[Key.Enter],
	() => {
	navigate(getSettingsPagePath(SettingsPath.NewServerlessFunction));
	},
	SettingsServerlessFunctionHotkeyScope.ServerlessFunction,
	);

[ehconitin]

Hello @Devessier!
This hotscope was introduced in #6656 , so are we supposed to revert it completely or find a better way to implement hotscope in serverless functions?

[Devessier]

Thanks @harshit078 and @ehconitin for your contribution. This issue has been solved.