Fix name field in Sprite import panel about litiengine HOT 13 CLOSED

Hey,

I am interested in resolving this issue. Would y'all be able to provide additional information and/or anything else that could help me?

from litiengine.

Hi, I'm sorry I was not yet able to respond. Having a busy week at work and some fun private events this week. I can't promise to be able to provide further details this week unfortunately.

from litiengine.

Thanks for your patience.
The problem lies in the SpritesheetImportPanel:

There is a JTextField that is supposed to show the name of the selected spritesheet. It is also supposed to let you change the spritesheet name with your input.
Neither of these behaviours work.

We might even consider dropping the JTextField entirely and letting users change the cell values in the sprite name list on the left.

from litiengine.

I suppose it's not going anywhere until then - no hurry here 🙂

from litiengine.

So I found a solution for the bug.

I got the Sprite name in the text field which updates the Sprite's name in the application.

from litiengine.

Saw this was still issue open but was resolved should be closed. Wasn't it supposed to close after the pull request was merged? Or did I not reference it properly?

If you add the text

Fixes #778 or Resolves #778 in a commit's message, it will close the issue when that commit is merged.

You can also put it anywhere in the first post of a pull request and it will also close the issue if the pull request is merged.

In the meantime though, we will have to wait for this to be closed manually by gurkenlabs.

from litiengine.

Saw this was still issue open but was resolved should be closed. Wasn't it supposed to close after the pull request was merged? Or did I not reference it properly?

If you add the text
Fixes #778 or Resolves #778 in a commit's message, it will close the issue when that commit is merged.
You can also put it anywhere in the first post of a pull request and it will also close the issue if the pull request is merged.
In the meantime though, we will have to wait for this to be closed manually by gurkenlabs.

Thank you so much. I just added Resolves #778 to the pull request.

It won't work now because it's already been merged.

from litiengine.

Hi again,

I just wanted to comment again about my interest in tackling this issue. Can y'all provide additional information and/or anything else that could help me? For example, how I can replicate the bug, what the proper functionality should look like, where in the utiLITI project should start looking in...etc.

I know how busy everyone is so please get back to me when it is most convenient for y'all.

from litiengine.

Awesome! I'm going to look into this bug sometime next week. (My schedule just got busy)

from litiengine.

Thank you for being patient with me. I just finished my courses for this Fall semester and have more free time now.
(It was brutal 😂)

So, I was trying to build Gradle in IntelliJ and got this error:

Build file 'C:\litiengine\build.gradle' line: 1

An exception occurred applying plugin request [id: 'de.gurkenlabs.semverguru', version: '1.3']
    > Failed to apply plugin 'de.gurkenlabs.semverguru'.
        > No match found

What do you think I should do to fix this?

from litiengine.

I believe this is #796 again. Have you made sure to compile with Java 21?

from litiengine.

Saw this was still issue open but was resolved should be closed. Wasn't it supposed to close after the pull request was merged? Or did I not reference it properly?

from litiengine.

Saw this was still issue open but was resolved should be closed. Wasn't it supposed to close after the pull request was merged? Or did I not reference it properly?

If you add the text

Fixes #778 or Resolves #778 in a commit's message, it will close the issue when that commit is merged.

You can also put it anywhere in the first post of a pull request and it will also close the issue if the pull request is merged.

In the meantime though, we will have to wait for this to be closed manually by gurkenlabs.

Thank you so much. I just added Resolves #778 to the pull request.

from litiengine.