Skip to content

Conversation

@codemist
Copy link
Collaborator

References:

Jira: MNTOR-5162
Figma:

Description

QA and @Vinnl found that while the breakpoint fix works in the guided resolution flow, it breaks the dashboard’s white background. I split the Monitor logo into its own component so I could use usePathname to apply the white bg on the image only on the main dashboard (not the resolution flow). This is a workaround for the two different views (dashboard vs. guided resolution flow).

Screenshot (if applicable)

image

How to test

Checklist (Definition of Done)

  • Localization strings (if needed) have been added.
  • Commits in this PR are minimal and have descriptive commit messages.
  • I've added or updated the relevant sections in readme and/or code comments
  • I've added a unit test to test for potential regressions of this bug.
  • If this PR implements a feature flag or experimentation, I've checked that it still works with the flag both on, and with the flag off.
  • If this PR implements a feature flag or experimentation, the Ship Behind Feature Flag status in Jira has been set
  • Product Owner accepted the User Story (demo of functionality completed) or waived the privilege.
  • All acceptance criteria are met.
  • Jira ticket has been updated (if needed) to match changes made during the development process.
  • Jira ticket has been updated (if needed) with suggestions for QA when this PR is deployed to stage.

@codemist codemist requested review from Vinnl and removed request for Vinnl January 26, 2026 15:27
@codemist codemist marked this pull request as draft January 26, 2026 15:28
@codemist codemist marked this pull request as ready for review January 26, 2026 16:08
args: {
type: "social-security-number",
},
parameters: {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Since MonitorLogo relies on usePathname and isn’t rendered with a real router context in the Storybook environment by default, the pathname is mocked at the story level (for all stories in the fix flow).

@codemist codemist merged commit 42732eb into main Jan 26, 2026
19 checks passed
@codemist codemist deleted the mntor-5162-fix branch January 26, 2026 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants