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

Fix font axis/feature SUI issues #17173

Merged
merged 2 commits into from
May 2, 2024
Merged

Fix font axis/feature SUI issues #17173

merged 2 commits into from
May 2, 2024

Conversation

lhecker
Copy link
Member

@lhecker lhecker commented May 1, 2024

Something something code changes, fixes issue. Events need throwing,
some events don't need throwing, some events need throttling to
not overwhelm the flurble function within the gumbies component.
This is all boilerplate and it works now.

Closes #17171
Closes #17172

Validation Steps Performed

  • Resetting the font axis/feature expander keeps
    the add new button flyout sorted ✅
  • Changing a font axis/feature value updates the preview ✅
  • After changing the font, features/axes can still be edited ✅

@@ -1420,6 +1421,8 @@ PUCHAR
pvar
pwch
PWDDMCONSOLECONTEXT
Pwease
Copy link
Member

Choose a reason for hiding this comment

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

LMAO

}

std::ignore = fontSettingsUser.Insert(std::wstring_view{ tagString }, value);
// Pwease call Profiles_Appearance::_onProfilePropertyChanged to make the pweview connyection wewoad. Thanks!! uwu
Copy link
Member

Choose a reason for hiding this comment

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

i am dying

@carlos-zamora carlos-zamora added this pull request to the merge queue May 1, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 2, 2024
@lhecker lhecker added this pull request to the merge queue May 2, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks May 2, 2024
@lhecker lhecker added this pull request to the merge queue May 2, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks May 2, 2024
@zadjii-msft zadjii-msft added this pull request to the merge queue May 2, 2024
@DHowett DHowett removed this pull request from the merge queue due to a manual request May 2, 2024
@DHowett DHowett merged commit 475b387 into main May 2, 2024
20 checks passed
@DHowett DHowett deleted the dev/lhecker/17171-sui-fixup branch May 2, 2024 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants