Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adaptive Tutor (WIP) #847

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

MischaU8
Copy link
Contributor

Proof of concept of a Rimworld-style Adaptive Tutor / Learning Helper.

Screenshot 2024-03-19 at 17 21 37

@SpacialCircumstances
Copy link
Member

I really like the concept of this, and it works fairly well already!

I have no qualms with the code so far, it looks clean and reasonably implemented to me.

@MischaU8
Copy link
Contributor Author

Based on some feedback, for this to be useful we need a way to highlight relevant UI elements.

@SpacialCircumstances
Copy link
Member

Based on some feedback, for this to be useful we need a way to highlight relevant UI elements.

I think the simplest approach would be the following:

  • Give every component that could be highlighted a descriptive ID (for example "carrier-build-button")
  • In the vue store, store a list of IDs that should be highlighted
    • The learning helper can then add IDs to that list
  • In the component, check if it should be highlighted and apply the required styles. This could be simplified by a helper function or a mixin

It doesn't feel too bad code-wise to me and should be fairly extensible/maintainable.

Mostly, this would leave the question of "when should a highlighting be removed?". That could either be when using that UI element, or be managed by the tutorial window, not sure what would make the most sense with the tutorial, you probably know what fits better.

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