How do i make the Menu be closeable from the Menu Button instead of only from the X button

Here should be your answer:

local Button = script.Parent:WaitForChild("TextButton")
local Frame = script.Parent:WaitForChild("Frame")

local On = false

local function OpenClose()
	if not On then 
		Frame.Visible = true
		On = true
	else
		Frame.Visible = false
		On = false
	end
end

Button.MouseButton1Click:Connect(OpenClose)

Locations:
https://gyazo.com/b3ab8dd68b346daa4accd7a60d36b3f6

it doesnt work.

it always says this:

Visible is not a valid member of ScreenGui “Players.JToHisveryfun.PlayerGui.Menu” - Client - MenuOpenAndCloseScript:4

You need to identify the MenuFrame, not the Menu. The Menu is a ScreenGui and doesn’t have that property.