-
-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
DOCS: update announcement bar #5193
Conversation
❌ Deploy Preview for mermaid-js failed.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5193 +/- ##
==========================================
+ Coverage 76.57% 76.60% +0.03%
==========================================
Files 164 164
Lines 13820 13820
Branches 693 693
==========================================
+ Hits 10582 10587 +5
+ Misses 3080 3077 -3
+ Partials 158 156 -2
Flags with carried forward coverage won't be shown. Click here to find out more. |
class="unstyled text-white plausible-event-name=bannerClick" | ||
class="unstyled flex-grow tracking-wide" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is plausible-event-name
being removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inadvertent removal. Fixed, thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The link checker CI failures aren't related to your PR, so everything looks good to me!
📑 Summary
This PR contains updates to the Announcement bar.
Resolves # n/a
📏 Design Decisions
Design decisions made by the Mermaid Chart team.
📋 Tasks
Make sure you
MERMAID_RELEASE_VERSION
is used for all new features.develop
branch