Skip to content

Fix all the Must Fix starting defold 1.10.0#663

Merged
AGulev merged 3 commits into
masterfrom
docs-actualization
Jul 13, 2026
Merged

Fix all the Must Fix starting defold 1.10.0#663
AGulev merged 3 commits into
masterfrom
docs-actualization

Conversation

@AGulev

@AGulev AGulev commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@AGulev
AGulev requested a review from britzl July 12, 2026 20:16

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 50211648ec

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/en/manuals/gui.md Outdated

*Max Dynamic Textures*
: The maximum number of textures that can be created using [`gui.new_texture()`](/ref/stable/gui/#gui.new_texture:texture_id-width-height-type-buffer-flip)
: The maximum number of dynamic textures tracked by this GUI component, `128` by default. This includes textures created with [`gui.new_texture()`](/ref/stable/gui/#gui.new_texture:texture_id-width-height-type-buffer-flip) and external textures assigned to the GUI with `go.set(..., "textures", ...)` or `gui.set(..., "textures", ...)`. Projects that replace many external textures may need to raise this limit.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Remove the nonexistent gui.set texture API

When readers try to replace external GUI textures using this new guidance, gui.set(...) is not a GUI Lua API, so that call will fail at runtime instead of updating the component. The supported component property path here is the go.set(..., "textures", ...) form (or the existing GUI texture-specific APIs), so listing gui.set gives users an unusable migration path.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AGulev
AGulev merged commit 53809c0 into master Jul 13, 2026
3 checks passed
@AGulev
AGulev deleted the docs-actualization branch July 13, 2026 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants