-
Notifications
You must be signed in to change notification settings - Fork 0
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
Mwpw 158926 company name wrap #82
Conversation
…ap long names to next line)
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## stage #82 +/- ##
=======================================
Coverage 83.57% 83.57%
=======================================
Files 10 10
Lines 3179 3179
=======================================
Hits 2657 2657
Misses 522 522 ☔ View full report in Codecov by Sentry. |
@@ -24,6 +24,13 @@ | |||
transform: translate3d(0,0,0); /* Fix Safari issues w/ position: sticky */ | |||
} | |||
|
|||
/*MWPW-158926*/ | |||
.feds-profile-menu { |
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.
Solution is good, but I think we should keep the way we write the css selectors consistent
.feds-profile-menu .company-placeholder
@SonjaPopovic I think the "before" should be the URL on the stage branch, not partner-stage.adobe.com |
before:
https://stage--dme-partners--adobecom.hlx.page/channelpartners/home/
after: https://mwpw-158926-company-name-wrap--dme-partners--adobecom.hlx.page/channelpartners/home/?georouting=off