-
Notifications
You must be signed in to change notification settings - Fork 522
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
Noticeboard refactor port + approved stamp #5283
Conversation
It still requires pens right? Could you map some in too while at it? As it stands reqs has paper and now the notice board but no pens |
Added a few |
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.
mapping lgtm
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.
nvrmind i'm blind, Semiotic symbols, you forgot to add them back
there is simply not much place for them |
Mapping was approved after changes requested were resolved.
About the pull request
Ports noticeboard refactor from TG, also i've made a third button to be able to interact with paper on noticeboard with a pen, in order to write on it, mapped noticeboard in req, as well as added some pens to req area.
Also added green approved stamp, added it in QM's office.
Explain why it's good for the game
Just compare these two, i think it says it all.
Testing Photographs and Procedure
Screenshots & Videos
Changelog
🆑
add: added approved stamp
fix: fixed being unable to write on papers on noticeboard
ui: changed noticeboard UI from scratch
refactor: refactored noticeboard code
maptweak: added noticeboard to req and approved stamp to QM's office, as well as some pens in req area to interact with noticeboard
/:cl: