Skip to content

Conversation

@mentatbot
Copy link

@mentatbot mentatbot bot commented Jun 27, 2024

This update introduces a new meta tag for specifying a custom skybox URL. The meta tag allows users to set a cube map URL, which is then incorporated into the render loop using native three.js functionality. The existing skybox behavior is disabled when a custom skybox is specified. This enhancement addresses issue #60.

Closes #60

This update introduces a new meta tag for specifying a custom skybox URL. The meta tag allows users to set a cube map URL, which is then incorporated into the render loop using native three.js functionality. The existing skybox behavior is disabled when a custom skybox is specified. This enhancement addresses issue #60.
@mentatbot mentatbot bot requested a review from gfodor June 27, 2024 15:58
@gfodor
Copy link
Contributor

gfodor commented Jun 27, 2024

A good start - please review dom-utils.js for how to properly manage the meta tag state, particularly note how the terrain meta tags are dealt with since this would largely fall under the same umbrella.

Copy link
Contributor

@gfodor gfodor left a comment

Choose a reason for hiding this comment

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

A good start - please review dom-utils.js for how to properly manage the meta tag state, particularly note how the terrain meta tags are dealt with since this would largely fall under the same umbrella.

@gfodor
Copy link
Contributor

gfodor commented Jun 27, 2024

@MentatBot

Copy link
Author

@mentatbot mentatbot bot left a comment

Choose a reason for hiding this comment

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

I'll update the PR to manage the meta tag state properly by reviewing and aligning with the approach used in dom-utils.js, especially how terrain meta tags are handled.

Thanks for using MentatBot. Reply with feedback or to ask MentatBot to review other parts of the PR. Please give feedback with emoji reacts.

This update introduces a new meta tag for specifying a custom skybox URL. The meta tag allows users to set a cube map URL, which is then incorporated into the render loop using native three.js functionality. The existing skybox behavior is disabled when a custom skybox is specified.

Additionally, the code has been updated to properly manage the meta tag state by utilizing functions from `dom-utils.js`, aligning with how terrain meta tags are handled. This enhancement addresses issue #60.

Closes #60
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.

add a meta tag for a custom skybox

1 participant