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

Ports tgstation#58980: better Tooltip support #5881

Conversation

mordent-goonstation
Copy link
Contributor

About the PR

Why's this needed?

  • Upstream parity good.
  • Better tooltip implementation.

…tip; code tidy up of some touched interfaces (ChemDispenser, MixingDesk, PowerMonitor, SlotMachine)
@boring-cyborg boring-cyborg bot added E-Docs Automatically applied on any .md change. A-UI Modifies UI in some way. Automatically applied on a change to tgui/ labels Sep 2, 2021
@github-actions github-actions bot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Sep 2, 2021
@mordent-goonstation mordent-goonstation added C-Code-Quality Cleans up code, refactors things to be more readable or intuitive size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Sep 2, 2021
bet: wager,
})} />
</Box>
<Stack vertical>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Definitely some out-of-scope changes in here, did them while trying to work out some of the layout things for the popper replacements. Happy to revert and do in a separate PR if needed; one of the perks of being a dev is that I can do what I want.

@@ -1951,6 +1951,13 @@ __metadata:
languageName: node
linkType: hard

"@popperjs/core@npm:^2.9.1":
version: 2.9.3
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tgstation locked to 2.9.2 because I guess that was what was available at the time. I'm fine with ours being ahead by a patch version.

@mordent-goonstation mordent-goonstation merged commit 9d351da into goonstation:master Sep 3, 2021
@mordent-goonstation mordent-goonstation deleted the tgui-upstream/58980-tooltip branch September 3, 2021 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-UI Modifies UI in some way. Automatically applied on a change to tgui/ C-Code-Quality Cleans up code, refactors things to be more readable or intuitive E-Docs Automatically applied on any .md change. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants