Skip to content

#16 & #26 Audio and Editor - #49

Merged
Av3boy merged 5 commits into
devfrom
16-av-3d-audio
Mar 25, 2025
Merged

Av3boy merged 5 commits into
devfrom
16-av-3d-audio

Conversation

@Av3boy

@Av3boy Av3boy commented Mar 25, 2025

Copy link
Copy Markdown
Owner

#16 & #26 Audio and Editor

Contents

This PR is trying to resolve:

  1. The ability to play .wav files.
  2. Fix some annoying issues in the editor UI.

We resolve it by:

  1. Introduce required classes for playing audio and a tester into the editor.
    1.2. Added a const class for some common audio types for future context menu shenanigans.
  2. Add a pre-render method to execute before the actual render call to tell imgui to render the context menu in the correct position.
    2.1. Added auto resizing to the context menu.
    2.2. Added ability to see object properties in editor by clicking on object.

Checklist

  • I have merged the latest changes from main to my branch.
  • I have tested my changes and any affected components.
  • I have added the proper documentation about my changes
  • I have made sure there is no overlapping work.
  • I have discussed any / all issues brought up from code review.

@Av3boy Av3boy self-assigned this Mar 25, 2025
@Av3boy Av3boy added audio Changes related to the audio in the engine bug Something isn't working core The issue is related to the Core library. editor Changes related to the Editor enhancement New feature or request window Changes related to the windows of the engine labels Mar 25, 2025
@Av3boy
Av3boy merged commit 57bf8cf into dev Mar 25, 2025
@Av3boy
Av3boy deleted the 16-av-3d-audio branch March 25, 2025 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

audio Changes related to the audio in the engine bug Something isn't working core The issue is related to the Core library. editor Changes related to the Editor enhancement New feature or request window Changes related to the windows of the engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant