We should have a button that convert an instance in the world to a prefab automatically.

Summary:

When you try to edit the width or height of a TileGridComponent in the world view out of its bounds, it crash the editor.

What should happen:

It should not allow to go out of bounds, without crashing.

How to reproduce the issue:

https://github.com/isadorasophia/hellomurder/assets/94173512/8c382636-d84a-4f68-a7a1-f42fddd0fc19
https://github.com/isadorasophia/hellomurder/assets/94173512/33866940-cb0f-4085-8d88-7c2c280dc79f
https://github.com/isadorasophia/hellomurder/assets/94173512/fea96988-b3d4-4414-becd-90800d17ca99

Repro

1. Create a prefab A
2. Create an instance B from prefab A
3. Delete prefab A

Expected
Prefab B will become a prefab by itself. Prefab A will be deleted.

Actual
Prefab A will be deleted. Prefab B will become "corrupted" and will need to be manually deleted. Any world instance of prefab A will be deleted from the world.

Fix
The fix is the tricky part. I guess we should scan all possible instances of prefab A prior to deleting it? Do we want to delete all the instances from the world or make them a complete instance (no prefab attached)?

We should come back at this eventually.

i followed the steps for neocity express and when i press f5 it says

Currently, I manually run in my machine:

dotnet build Murder.sln
.\tools\dotnet2md.ps1 ..\src\Murder\bin\Debug\net7.0
tools\publish.ps1

So it outputs the md files at docs\src and the html files at docs\book. We should rather have a GitHub action (or a bot?) that automatically generates the html files.

The locked status of layers should be remembered after closing the editor. This should be stored on the EditorSettings, though, not on the WorldAsset, since it's a local preference, not a project one.

Check Grid.cs.
CellSize is hardcoded, the default value should be available to change on GameProfile and each scene should have it's own grid size.

And create all appropriate helpers as extensions.
Using System.Numerics.Vector2 will allow us to use hardware acceleration with vector math and will make things easier to integrate with other libraries.

Summary:

Editor crash when trying to reset the value of a tileset inside the TilesetComponent in world.

What should happen:

Editor shouldn't crash.

How to reproduce the issue:

https://github.com/isadorasophia/hellomurder/assets/94173512/96657bdd-0f29-467d-a5ec-2b1b005e3bcc
https://github.com/isadorasophia/hellomurder/assets/94173512/e7509cf4-0426-4d21-ab39-884a75402c3b

Sometimes the collider polygon shapes are showing with different sizes on the WorldAssetEditor the correct size can be seen both in-game and on the prefab editor.

The search box we have right now currently very functional, but has a few problems and could be improved:

• Opening with the mouse cursor far to the right will extend it, then it will slowly contract to a fixed size (?). This shouldn't happen. It should take the horizontal size of the field.
• Arrow keys should select an option and enter should choose it.
• Pressing escape should close it.

Check Murder.Editor\Utilities\Gui\SearchBox.cs for current implementation.

Good first issue for those familiar with ImGui

When importing linked cells that are not the first frame the Aseprite importer is skipping those.

It would be great to have some nice quality of life improvements on the entity selector. This would only be enabled if the entity is selected - otherwise, move camera, like we have it now:

• Arrow keys move entities by 1 pixel
• Shift+Arrow keys move entities by 1 tile.

Other features that would be nice:

• Hold shift while dragging constrain to an axis (horizontal, vertical).
• Alt+Click+Drag duplicate entity and drag.

We don't have an undo feature except on ImGui fields, we eventually will need one, specially for the map editor.
Maybe we can start simple and add more "undoable" things in the future?

The UI for fonts is broken.
This reproduces on all systems (Win/macOS/Linux).

How to reproduce:

1. clone this repo.
2. run dotnet run --project src\HelloMurder.Editor\HelloMurder.Editor.csproj
3. under Fonts click on any font.

/e under linux it's a bit worse

A little number on the side, like Unity, can work:

ImGui does not support clipboard by default in Linux and Mac. We already implemented Mac scenario, see this line:

private void InitializeImGui()
{
    // Magic so ctrl+c and ctrl+v work in mac!
    if (OperatingSystem.IsMacOS())
    {
        ImGuiIOPtr io = ImGui.GetIO();

        if (OperatingSystemHelpers.GetFnPtr is IntPtr getFnPtr)
        {
            io.GetClipboardTextFn = getFnPtr;
        }

        if (OperatingSystemHelpers.SetFnPtr is IntPtr setFnPtr)
        {
            io.SetClipboardTextFn = setFnPtr;
        }
    }
}

We should implement such that OperatingSystemHelpers also supports Linux.

When trying to sent entity agent impulse, the impulse never updates the movement of the agent, since it gets cleared first.

Notes:

• Working out of the HelloMurder project
• I manually added the PlayerInputSystem into the Systems of the World "Hello World" to register input
  

Order of execution:
PlayerInputSystem FixedUpdate()
entity.SetAgentImpulse( _lockMovement ? Vector2.Zero : _cachedInputAxis, direction);
Then AgentCleanupSystem FixedUpdate runs, removing the impulse
Therefore AgentMoverSystem Update never updates the velocity.

Workaround:
Commenting out / disabling AgentCleanupSystem allows the impulse to apply correctly, but then friction is never reapplied to the agent by the cleanup system.

Perhaps it has something to do with AgentMoverSystem using Update and not FixedUpdate, or otherwise the order of operations matters here?