Skip to content

Semantic Tokens: Tooltip - #34845

Merged
Mason Tejera (mltejera) merged 6 commits into
extended-tokensfrom
extended-tokenss/matejera/tool-tip
Jul 17, 2025
Merged

Semantic Tokens: Tooltip#34845
Mason Tejera (mltejera) merged 6 commits into
extended-tokensfrom
extended-tokenss/matejera/tool-tip

Conversation

@mltejera

Copy link
Copy Markdown
Contributor

The changes include adding new tooltip styles, integrating them into the semantic style hooks, and simplifying semantic token definitions by removing redundant values and unused tokens.

Tooltip Integration and Styles:

Semantic Token Refinements:

@mltejera
Mason Tejera (mltejera) requested a review from a team as a code owner July 16, 2025 00:11
@mltejera
Mason Tejera (mltejera) force-pushed the extended-tokenss/matejera/tool-tip branch from b5c8dfb to 42e3cf3 Compare July 16, 2025 17:18
Comment thread packages/semantic-tokens/src/fluentOverrides.ts
Comment thread packages/semantic-tokens/src/legacyVariant/tokens.ts Outdated
@etudie

Xue (etudie) commented Jul 16, 2025

Copy link
Copy Markdown

Looks great, possibly missing the build commit? yarn nx build semantic-style-hooks-preview

example: https://github.com/microsoft/fluentui/pull/34842/files#diff-03b609b8fcb6f7dee9c9ee2971f09eb254bfcf767c07c5faf76c45d33a38b2f7

@mltejera
Mason Tejera (mltejera) requested a review from a team as a code owner July 16, 2025 17:47
@github-actions

Copy link
Copy Markdown

Pull request demo site: URL

@mltejera
Mason Tejera (mltejera) force-pushed the extended-tokenss/matejera/tool-tip branch from 2e6924b to d53f51b Compare July 17, 2025 20:40
@mltejera
Mason Tejera (mltejera) removed the request for review from a team July 17, 2025 20:54
@mltejera
Mason Tejera (mltejera) merged commit 54dc2e6 into extended-tokens Jul 17, 2025
13 of 17 checks passed
Mitch-At-Work pushed a commit that referenced this pull request Aug 6, 2025
Mitch-At-Work pushed a commit that referenced this pull request Aug 20, 2025
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.

4 participants