Merged
Conversation
…ent (#1000) * Bug-1999558: Display complete commit message even when newline is absent This change ensures that single line commit messages with absent newlines are displayed correctly. If a newline is present in the message summary, then it is treated as the commit message. If no newline is present in the message summary, then the entire summary is taken as commit message. A new test has been added to verify the change. The snapshots have been regenerated to reflect the changes. @kala-moz @davehunt Fixes [Bug-1999558](https://bugzilla.mozilla.org/show_bug.cgi?id=1999558) * Bug-1999558: Implement review comment Implement review comment by making the added comment into a multiline comment. * Bug-1999558: Implement review comments Remove Bug number from comment as suggested in review comment * Bug-1999558: Fix merge conflicts Fix merge conflicts by rebasing code over latest changes in main.
* Bug-1915777: Make test tags theme aware This change updates the TestHeader and SubtestsRevisionHeader by making tag options aware of light. The changes made are: * Import theme hook in both TestHeader and SubtestsRevisionHeader * Update `styles` object in both TestHeader and SubtestsRevisionHeader to choose background color of span based on theme * Update color of tag based on theme * Add new colors to `styles/Colors.ts` * Bug-191577: Adding helpers for setting tag color Moved the logic to find the appropriate colors for the tags into helpers and used those helpers in the components. Also, added tests to validate the helpers. * Bug-1915777: Use Mui's theme aware <Box> element Implemented review comment and used Mui's theme aware `<Box>` element. Added the required options to `protocolTheme.ts`. Also, removed implementation added in previous commits as well as the tests and regenerated the spanshots by running `npm run fix-all`
⬆️ Update all npm dependencies (2026-02-20) Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
⬆️ Update all npm dependencies (2026-02-24) Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
* created test version registry and separate files for test versions Row template columns now only gets the columns for the version * expanded test strategy to include getting columns for versions and get avg values * update comments * updated comments explaining types
* remove replicates toggle for mwu test version * remove toggle for subtests view
⬆️ Update all npm dependencies (2026-03-09) Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
* updated mwu tooltips with more info and links * Update all npm dependencies (2026-03-09) (#1007) ⬆️ Update all npm dependencies (2026-03-09) Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com> * improved with note on sign and expand tooltip width * update snapshots * updated mwu tooltips with more info and links * improved with note on sign and expand tooltip width * update snapshots --------- Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com>
* Bug-1948879: Adding links to docs and source in banner Added links to PerfCompare docs and GitHub source in the Banner. * Bug-1948879: Update snapshots Update snapshots * Bug-1948879: Fix snapshots Fix snapshots. * Bug-1948879: Added a shared component for Links Added a shared `NavBarLink` component for the below: - Docs - Contact - Source * Bug-1948879: Added the links to Docs, GitHub, Matrix and Bugzilla in the header Made the below changes to add new links to urls that were previously in Banner * Created a new NavBarLink component as a single reusable component for Links in the Header * Updated the PerfCompareHeader component to add a new Box with NavBarLinks - Docs, Source, File Bug, Matrix * Updated the Strings to contain the required strings for tooltip The Link font styles are kept identical to the Dark Mode switch label styles to ensure a consistent UX. The Link is also theme aware. * Bug-1948879: Removed the Banner and updated tests Made the below changes to remove the banner: * Updated the PerfCompareHeader to remove the Banner in the `<Alert />` component * Removed the `Banner` component * Updated the tests for `App.tsx` to check for all links and their correct tooltip * Bug-1948879: Updated test for rendering page Updated the tests for rendering page by: * Made the test for checking heading more specific * Checked for entire Perfcompare tagline text * Bug-1948879: Updated Snapshots Executed linter and updated snapshots * Bug-1948879: Implement Review Comments * Removed additional styling in NavBarLink's `<Box>` * Corrected spacing in the PerfCompareHeader * Bug-1948879: Implement Review Comments Deleted the lines related to Mode switching from NavBarLink.tsx
* ⬆️ Update all npm dependencies (2026-03-13) * updated snapshots --------- Co-authored-by: depfu[bot] <23717796+depfu[bot]@users.noreply.github.com> Co-authored-by: Carla Severe <csevere@mozilla.com>
kala-moz
requested review from
beatrice-acasandrei,
canova and
esanuandra
as code owners
✅ Deploy Preview for mozilla-perfcompare ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Check out the change-log below to see the updates:
[kala-moz]
Test version refactor : column config and rendering (#1004)
BUG 2019125: Remove replicates toggle for mwu test version (#1006)
BUG 2017544: update MWU tooltips with more info and links (#1008)
[moijes12]
Bug-1999558: Display complete commit message even when newline is absent (#1000)
Bug-1915777: Make test tags theme aware (#993)
Bug-1948879: Add links to docs and source in the Banner (#995)