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(toolbar): Fixes small misc issues #10442

Merged
merged 5 commits into from Mar 15, 2024
Merged

fix(toolbar): Fixes small misc issues #10442

merged 5 commits into from Mar 15, 2024

Conversation

Princesseuh
Copy link
Member

@Princesseuh Princesseuh commented Mar 14, 2024

Changes

This fixes two small issues that got reported after 4.5 with the audit panel:

  • If the lints refreshed while the panel was open, the panel wouldn't update correctly
  • If a lint contained HTML tags, they would get wrongly stripped from the description in the panel. We now have a small Markdown support like we do for the error overlay, so we can show things nicely while still generally escaping HTML

image

Testing

Updated mutation test to test for the audit UI as well

Docs

N/A

Copy link

changeset-bot bot commented Mar 14, 2024

🦋 Changeset detected

Latest commit: 170c7fe

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Mar 14, 2024
@Princesseuh Princesseuh marked this pull request as ready for review March 15, 2024 11:12
@Princesseuh Princesseuh merged commit f8e0ad3 into main Mar 15, 2024
13 checks passed
@Princesseuh Princesseuh deleted the fix/misc-devtoolbar branch March 15, 2024 12:26
@astrobot-houston astrobot-houston mentioned this pull request Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: astro Related to the core `astro` package (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants