godotengine / godotengine/godot-docs

NOTIFICATION_WM_CLOSE_REQUEST is incorrectly described to exist in MainLoop by some documentation.

Open
#8,891 0 comments 0 reactions 0 assignees View on GitHub
area:class reference area:manual bug
Dominant language
reStructuredText
Stars
5.7k
Forks
3.8k
Avg merge
1d 20h
Merged PRs (30d)
25

Description

**Your Godot version:** v4.2.stable.official

**Issue description:**
`MainLoop.NOTIFICATION_WM_QUIT_REQUEST` and `MainLoop.NOTIFICATION_WM_CLOSE_REQUEST` are referenced, but neither one works (despite the phrasing "you _can_ remove the MainLoop. prefix" implying that it is optional); the only one that works is `NOTIFICATION_WM_CLOSE_REQUEST`.

`MainLoop` class documentation correctly _excludes_ both but does not direct the user to the correct location of these constants; maybe they were removed from the class unintentionally?

**URL to the documentation page:**
https://docs.godotengine.org/en/stable/tutorials/migrating/upgrading_to_godot_4.html
> MainLoop's NOTIFICATION_ constants were duplicated to Node which means you can remove the MainLoop. prefix when referencing them.
> MainLoop's NOTIFICATION_WM_QUIT_REQUEST is now NOTIFICATION_WM_CLOSE_REQUEST.

https://docs.godotengine.org/en/stable/classes/class_mainloop.html

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.