feat(#17): add notification on new skill learned

[MrGizmo123]

added a popup notification when a new skill is acquired as per #17

Closes #17

[jdrueckert]

First of all: thanks a lot for this PR, tests out like a charm ✨
Second, is there any particular reason, why you committed your logback-test.xml? If not, please remove it from the PR.

[jdrueckert]

Please remove this unused import

Suggested change

import org.terasology.engine.logic.players.event.LocalPlayerInitializedEvent;

[jdrueckert]

I think we can omit the repetition of "New Skill Acquired" in combination with "Acquired Skill".
How about

Suggested change

	"New Skill Acquired",
	"Acquired Skill " + newSkillName,
	"Today I Learned",
	newSkillName,

[keturn]

module get supplies logback-test.xml, and it's not git-ignored because having it in the repo lets you adjust how things show up on CI. I expect all modules will end up with one. If that sounds like a bad idea, we should talk about that (in some appropriate venue).

[MrGizmo123]

This is actually my first PR so I'M kind of new to this. Will remove the XML

[jdrueckert]

module get supplies logback-test.xml, and it's not git-ignored because having it in the repo lets you adjust how things show up on CI. I expect all modules will end up with one. If that sounds like a bad idea, we should talk about that (in some appropriate venue).

💡 In that case, let's try to get a suitable version of it with the configuration we want in CI in for all omega modules @keturn
With @skaldarnar's workspace tooling that should be a fairly quick change.

[jdrueckert]

This is actually my first PR so I'M kind of new to this. Will remove the XML

No worries, all good, you're doing great 👍
Many first-time PRs we get don't even work or implement the desired change, but yours does that and tests out great 😃