-
Notifications
You must be signed in to change notification settings - Fork 117
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
Add Winter holiday event models to the lobby #768
Conversation
Tree model from r5 and modified by 4Volt other models are from 4Volt Co-authored-by: EM4Volts <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes mostly look good, just some incorrect formatting.
I'm not a fan of the multiple rpaks approach, but this has been discussed on discord (4v doesn't want to use the RePak refactor until it has a github release)
Northstar.Custom/mod/scripts/vscripts/ui_ns_custom_mod_settings.gnut
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apart from formatting, this looks good.
I could test this by modifying the date, and verify new models appear in the lobby.
Private lobby menu is however hard to read white-on-white, I don't know how this could be fixed (rotating the camera upwards maybe? Adding another present just below menu position?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(bikesheddding)
A bunch of newly added files are missing trailing newlines
Northstar.Custom/mod/scripts/vscripts/ui_ns_custom_mod_settings.gnut
Outdated
Show resolved
Hide resolved
i think blue tried looking into moving cam or something. in the end we decided on sticking with this. since one menu is usable in other menus aswell, and the other is the store which isnt usable anyway. its a small time window and u can also still use the menu if u hover over it as the button label colors change then. |
you should be able to edit the camera position with by moving |
the camera is set through animations so we have 2 Options |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good enough, missing some localisation but this is a bit time sensitive so yeah
i dont think changing it would be bad. maybe we can sit together in the coming days and work out a solution for it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good
"needs testing" just tested by changing the dates in script to trigger early. works. |
Missing localisations are easier to add post-merge actually as they are picked up by weblate which is way more accessible to non-developers than GitHub anyway ^^ |
Nah, because like its just using string literals, so it can't be localised atm. We can still just fix this later very easily but yeah |
Northstar.Custom/mod/materials/models/northstartree/lightsflicker.vtf
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requests from reviews have been addressed.
Models will be shown from 18.12. to the 06.01.
the tree model is from r5 and modified by 4Volt
The other models are from 4Volt
Co-authored-by: @EM4Volts