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
TGUI Search component and disposal / mail chute UI #1844
Conversation
…nto disposal-tgui
Co-authored-by: Mordent <62817778+mordent-goonstation@users.noreply.github.com>
…into disposal-tgui
probly want the text box thing fixed b4 merge |
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.
Approving with caveat that someone (possibly me) fixes the issue with the Input
component first.
…nto disposal-tgui
blocking this due to the input thing |
…nto disposal-tgui
I've redone the design on this to have a look and feel closer to that of the other UIs that I have designed Because the code was changed so significantly in the process I am re-requesting a review from Mordent |
…nto disposal-tgui
…nto disposal-tgui
…nto disposal-tgui
About the PR
edit: I've redesigned my initial submission significantly to a newer cleaner feel
Images below are of previous revision
NOTE: the Disposal Handle label in the images has been renamed Chute Handle, very small change so i didnt make new pics
https://user-images.githubusercontent.com/33204415/90360958-062c3f00-e02b-11ea-9901-c078c8f66c4c.png
https://user-images.githubusercontent.com/33204415/90360991-16441e80-e02b-11ea-848c-13fbcac33d9f.png
https://user-images.githubusercontent.com/33204415/90361022-2825c180-e02b-11ea-8ef4-87a005e21dc6.png
https://user-images.githubusercontent.com/33204415/90361079-4f7c8e80-e02b-11ea-87ee-be95da7f59ce.png
https://user-images.githubusercontent.com/33204415/90361144-75099800-e02b-11ea-991e-48f8702b55b8.png
There is a known issue with text input fields that should be fixed before this is merged. While it is functional as is, the user experience would be poor when searching for mail destinations.
Why's this needed?
Changelog