Wrong Backgrounds #25
Labels
No Label
bug
Critical Priority
duplicate
enhancement
help wanted
High Priority
Low Priority
Medium Priority
Meta
needs more info
question
refactor
wontfix
bug
Critical Priority
duplicate
enhancement
help wanted
High Priority
invalid
Low Priority
Medium Priority
Meta
question
wontfix
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Cavemanon/SnootGame#25
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
At least one person is reporting Backgrounds that do not seem to match what the text says.
They seem to indicate that it's an issue mostly before the initial rooftop scene.
Listed were
I have not personally double checked that these are errors, and will look into it tonight. Just wanted to create an issue when it got brought up in thread.
Starting to dig into this now.
Edit: Looked into the initial 3 issues anon reported haven't gone through the whole game yet. Lot's of speculation ahead.
Lines ~4651 ~6643
Both times the texts supports you being in the Auditorium, but the background is the listed bandroom. Possibly used as a placeholder for an asset never created of the auditorium stage, should be an easy fix to change the file pointers
Line ~7773
Anon leaves his house with a fade to get on a bus and is instead inside Fang's house already. Again possibly a placeholder, or just needs removal to continue the fade or possibly use the sidewalk BG.
@PrincipleSpears please let me know if I am wrong about any of this, this isn't a code issue as much as a story/knowledge issue. If the above assumptions are correct we can just roll the this into my intial PR, unless you would prefer them seperately for some reason.
Backgrounds not matching text is a code issue (after all they've been coded in) so this can stay. If you want to roll this into a PR, sure, but make sure that you note it down as solving this issue as well.
Better explanation. I understand it is literally a code issue (especially the fade scene), but as someone who's highschool auditorium always had band stuff on stage, I could see a wierd reality where it being the background for those scenes made sense.
But of course I'll mark it as such
Edit: PR is up and should fix this issue
Ghost referenced this issue2021-06-18 01:13:51 +00:00
do you mind making a version of this PR for when https://git.snootgame.xyz/Cavemanon/SnootGame/pulls/17 gets merged? It'll be soon enough.
Of course. Sort of assumed with the big boy almost being done that would be the case. I am currently on the road, but I will crank this out when I get to my destintation. As a heads up I will be unavailable tomorrow untill probably ~1800 CDT.
sweet, can't wait to see it.
PR has been updated. Not tested. I'm doing this on a laptop in the back of a truck, using a mobile hotspot lol so apologies if something is borked.
Back of truck testing seems to work as intended.