Skip to content

Basic messages with currently buggy support of BBCode#7

Open
Sulucnal wants to merge 4 commits intoMaruno17:mainfrom
Sulucnal:basic-messages
Open

Basic messages with currently buggy support of BBCode#7
Sulucnal wants to merge 4 commits intoMaruno17:mainfrom
Sulucnal:basic-messages

Conversation

@Sulucnal
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Collaborator

@NuriYuri NuriYuri left a comment

Choose a reason for hiding this comment

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

That's a big PR so I will need to come back later for other details. I could not test yet.

@onready var main : Main = get_tree().root.get_child(-1)


var last_scene_added : Variant
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What's the purpose of that variable?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It is used by the scene manager to add and remove nodes from the main scene.

await get_tree().create_timer(GlobalVar.milliseconds_per_char).timeout


func _process(_delta: float) -> void: #I don't like the way it's done but I'm too tired to solve it right now.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Don't hesitate to discuss with voxet if you need support on that. They showed a battle video they implemented and there's a message showing in there :)

@NuriYuri
Copy link
Copy Markdown
Collaborator

Hi, this PR has code related to that PR: #8

Can you both sync together to make sure each PR does one thing and does not conflict with each other?

@NuriYuri
Copy link
Copy Markdown
Collaborator

NuriYuri commented Apr 5, 2025

Hi :)

I see that the PR for Godot 4.4 was merged. Can you attempt to rebase and see how this PR behave in regard of Godot 4.4? (If you have time)

@Sulucnal
Copy link
Copy Markdown
Contributor Author

Sulucnal commented Apr 5, 2025

Hi :)

I see that the PR for Godot 4.4 was merged. Can you attempt to rebase and see how this PR behave in regard of Godot 4.4? (If you have time)

Regarding that, I have a problem regarding conflicts with this PR: #8. I left them a message but they probably didn't see it or got busy. I'll try to get in contact with them to see if they can either do what I asked of them or let me use a part of their code that handled a problem I had in a better way than what I did.

@NuriYuri
Copy link
Copy Markdown
Collaborator

NuriYuri commented Apr 5, 2025

Ok :)

Tell me if you need a decision or something in few days :)

@NuriYuri
Copy link
Copy Markdown
Collaborator

I closed #8 for the lack of answer.
You can try to take what best from that PR and maybe improve this using stuff like: https://godotengine.org/asset-library/asset/3654

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.

2 participants