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

Tooltip visual updates #4077

Merged
merged 9 commits into from
Feb 2, 2021
Merged

Tooltip visual updates #4077

merged 9 commits into from
Feb 2, 2021

Conversation

tashatitova
Copy link
Contributor

@tashatitova tashatitova commented Feb 2, 2021

  • Updated brushes (inverse style), corner radius and padding.
  • Removed unnecessary corner radius before contract7 part.

tooltip-light

tooltip-dark

tooltip-hc

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Feb 2, 2021
@tashatitova
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tashatitova
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mdtauk
Copy link
Contributor

mdtauk commented Feb 2, 2021

Why was the choice of using an inverted Tooltip reversed? It did help them stand out better from the rest of the UI

@tashatitova
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tashatitova
Copy link
Contributor Author

Why was the choice of using an inverted Tooltip reversed? It did help them stand out better from the rest of the UI

Legacy tooltips won't be updated to inverse so they won't be uniform.

@mdtauk
Copy link
Contributor

mdtauk commented Feb 2, 2021

Why was the choice of using an inverted Tooltip reversed? It did help them stand out better from the rest of the UI

Legacy tooltips won't be updated to inverse so they won't be uniform.

Ah, yes that would be a problem. Shame that the older ones wont be changed.

@tashatitova
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tashatitova
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@tashatitova tashatitova merged commit 6b095b1 into master Feb 2, 2021
@tashatitova tashatitova deleted the user/tatito/tooltip branch February 2, 2021 21:16
@StephenLPeters StephenLPeters added area-Styling area-ToolTip area-UIDesign UI Design, styling team-Controls Issue for the Controls team and removed needs-triage Issue needs to be triaged by the area owners labels Feb 4, 2021
Kinnara added a commit to Kinnara/ModernWpf that referenced this pull request Nov 15, 2022
Kinnara added a commit to Kinnara/ModernWpf that referenced this pull request Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Styling area-ToolTip area-UIDesign UI Design, styling team-Controls Issue for the Controls team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants