-
Notifications
You must be signed in to change notification settings - Fork 37
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
Release tracking #861
Release tracking #861
Conversation
❗ Pre-merge checklistPlease ensure these items are checked before merging. ✅ Preliminary checks
🔌 Integration smoke testsDotcom
Subdomain sites
Sandboxes
🤔 Cross-check (aka sanity test)
After tests
🚢 After merge
|
🟢 No design token changes found |
🟢 No visual differences foundOur visual comparison tests did not find any differences in the UI. |
4dc6abe
to
9c82d93
Compare
a2b6779
to
b66d140
Compare
b66d140
to
055bde6
Compare
055bde6
to
d01df0a
Compare
d01df0a
to
153e7e4
Compare
Smoke test results
Component works as expected Release testing status: Ready to release 🟢
Default heading size verified as being reduced to Release testing status: Ready to release 🟢
Actions are smaller, and variant gets forwarded as expected Release testing status: Ready to release 🟢
Links in SubMenu can now be marked as current, and style improvements are present. New Release testing status: Ready to release 🟢
Release testing status: Ready to release 🟢
New variants work as expected Release testing status: Ready to release 🟢 githubuniverse.comThis release of @primer/react-brand triggers a new warning in the terminal when running githubuniverse.com locally.
This is being triggered by this line in the SubNav which was introduced in 21b2584. We're using Resources HubThe same issue ( |
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
@primer/[email protected]
Minor Changes
#860
2502f658
Thanks @rezrah! - Decreased the default heading size inRiver
component. Usesize="4"
to restore the previous text size, if needed.#863
00ef0694
Thanks @rezrah! - Added a newFrostedGlassVFX
component for applying a frosted glass-effect texture to nested components.When using this component, ensure there is sufficient contrast between the foreground text and the background imagery, as the frosted glass effect can significantly reduce legibility.
Usage example:
🔗 (See Storybook for an example)[https://primer.style/brand/storybook/?path=/story/components-testimonial-examples--with-frosted-glass]
Patch Changes
#859
cd18615f
Thanks @rezrah! -AnchorNav
component updates:AnchorNav.Action
andAnchorNav.SecondaryAction
now appear visually smaller by default. Usesize="medium"
if the previous, larger buttons are needed.AnchorNav.Action
andAnchorNav.SecondaryAction
each support avariant
prop, allowing primaryButton
visuals to be optionally applied.#866
8223135d
Thanks @rezrah! - Fixed delayed appearance of heading separator inSubNav
.#865
0401cc5c
Thanks @joshfarrant! - Style improvements to theSubNav
component.SubNav.Link
elements witharia-current
, which caused inconsistent appearance across viewports.#863
00ef0694
Thanks @rezrah! - Added newTestimonial
variants.Use
variant="default"
orvariant="subtle"
for an alternative visual appearance. The current design will be referred to asminimal
going forward.🔗 See the documentation for usage examples
#867
4bac3868
Thanks @rezrah! -Hero.Description
updated to supportvariant="muted"
#868
c0aa598d
Thanks @rezrah! -SubNav.Link
now supportsvariant="default"
to help increase contrast and legibility on non-standard backgrounds.@primer/[email protected]
Patch Changes
#863
00ef0694
Thanks @rezrah! - Added new design tokens forTestimonial
andFrostedGlassVFX
components@primer/[email protected]
@primer/[email protected]
@primer/[email protected]
@primer/[email protected]