Skip to content

Conversation

@tvst
Copy link
Contributor

@tvst tvst commented Sep 3, 2021

(Breaking up #3642 into multiple PRs)

  • Set tooltip background to match the primary background color instead of the secondary one
  • Improve code that sets tooltip font to sm, so it works for all inner children

NOTE: Screenshot tests coming after first round of reviews. Don't want to generate and re-generate as more reviews come in 😅

NOTE #2: Note the base branch for this PR. I'm basing it on the previous UI tweaks PR so the diff is easier to read. The idea is to merge in order.

@tvst tvst requested a review from vdonato September 3, 2021 05:59
@tvst tvst force-pushed the feature/ui-8-statuswidget branch from 39cc561 to a8b1975 Compare September 5, 2021 16:18
@tvst tvst force-pushed the feature/ui-9-tooltip branch from 7e17629 to 14c5137 Compare September 5, 2021 16:22
Base automatically changed from feature/ui-8-statuswidget to feature/ui-updates September 5, 2021 16:27
@tvst tvst merged commit 55f35b6 into feature/ui-updates Sep 6, 2021
@tvst tvst deleted the feature/ui-9-tooltip branch September 6, 2021 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants