ScalarVector1 / DragonLens

Modern, customizable, and community-driven debugging and cheat mod for TModLoader
24 stars 14 forks source link

Make "Advanced Layout" provide everything + Add Tile Remover #57

Closed Setnour6 closed 1 year ago

Setnour6 commented 1 year ago

This issue leans towards a feature request. Using Windows 11, Steam, Latest 1.4.4 version

Issue 1 "Advanced Layout" Description

Clicking on the Advanced Layout button in the Layout Presets UI does not present the following options:

See the below screenshot: image

The "Clear Buffs" option could be on the right of the screen where the rest of the removal functions/options are.

Issue 2 "Tile Remover" Description

There is only an option for adding tiles via the "Tile Spawner". There are some tiles that cannot be removed through normal means, or if a tile cannot be destroyed by a pickaxe of any level. I would like to see a "Tile Remover" option added as well (unless Tile Painter does the same thing)

I have yet to see whether Tile Painter also copies the air (or if there is a toggle for copying it). The game freezes anyways.

There could be a toggle that tells you to "remove tiles" and it would function the same way as placing a selected tile, but you are instead erasing the tile.

ScalarVector1 commented 1 year ago

Tile painter will copy air and works for the purpose, though I could see the application for a simple 'tile zapper'. I do wonder if adding an 'air' option to the tile spawner may fill this use though without an entire new tool?

Setnour6 commented 1 year ago

Tile painter will copy air and works for the purpose, though I could see the application for a simple 'tile zapper'.

I tested it, and air is also copied.

I do wonder if adding an 'air' option to the tile spawner may fill this use though without an entire new tool?

That could work. the toggle could make it so when you copy tiles and you accidentally (or not) include the air, pasting the structure will not have the air overwrite existing tiles. Example: you have two houses seperated from each other and you want to copy and paste them into a cave. The toggle could be set to 'off' and that would make it so the dirt walls in the cave (if you paste there) will not be affected by also pasting the air.

I did also notice that I can't change my mind about painting tiles, in the case of right-clicking to stop the tile selecting square when I change my mind about selecting something. I can make a seperate issue about that.

ScalarVector1 commented 1 year ago

Example: you have two houses seperated from each other and you want to copy and paste them into a cave. The toggle could be set to 'off' and that would make it so the dirt walls in the cave (if you paste there) will not be affected by also pasting the air.

Null tiles in StructureHelper cover this use case, though that ofcourse requires the installation of StructureHelper to be able to place down

Setnour6 commented 1 year ago

Example: you have two houses seperated from each other and you want to copy and paste them into a cave. The toggle could be set to 'off' and that would make it so the dirt walls in the cave (if you paste there) will not be affected by also pasting the air.

Null tiles in StructureHelper cover this use case, though that ofcourse requires the installation of StructureHelper to be able to place down

I was going to say that you could add the tiles from StructureHelper to DragonLens. There will be two tiles that do the exact same thing though. I could try to enable StructureHelper and see what I can do in this scenario

Setnour6 commented 1 year ago

Hello, it's been almost two months. Issue 1 is still prominent, given that when I loaded the latest workshop version of DragonLens, the Buff Despawner and newly added Sound Player tools do not appear at all. Right now, the advanced layout looks like this: image

If I recall, the toolbars at the bottom should also be spaced out.

ScalarVector1 commented 1 year ago

You likely need to clear out your old saved layout directory. Deleting the tModLoader/DragonLensLayouts directory should ensure new layouts are generated properly.

Setnour6 commented 1 year ago

You likely need to clear out your old saved layout directory. Deleting the tModLoader/DragonLensLayouts directory should ensure new layouts are generated properly.

I cleared the directories for both preview and non-preview builds of tModLoader. It works, so part of this issue is solved. I'll close this issue anyways since there hasn't been any activity for the tile remover part of this issue (I hadn't had the chance to test it)