Skip to content
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

feat(Masthead): React masthead alternate logo and tooltip options #4401

Merged
merged 14 commits into from
Nov 5, 2020
Merged

feat(Masthead): React masthead alternate logo and tooltip options #4401

merged 14 commits into from
Nov 5, 2020

Conversation

kennylam
Copy link
Member

@kennylam kennylam commented Nov 4, 2020

Related Ticket(s)

#3906

Description

The masthead now accepts an optional logo options from IBM.com translation files which allows the setting of an alternate IBM logo to use, as well as an optional tooltip when the icon is hovered over. When an alternate logo is used, settings include an expiration date and allow/deny lists for controlling paths on which it can be displayed. The tooltip can be used with both default and alternate logos.

Changelog

New

  • MastheadLogo service to determine whether an alternate logo can be used based on values passed in the object
  • React Masthead component now accepts options from IBM.com translations for an alternate logo and tooltip
  • internal ConditionalWrapper component

Changed

  • add masthead logo svg dimension styles

@kennylam kennylam changed the title Feat/custom masthead logo feat(Masthead): React masthead alternate logo and tooltip options Nov 4, 2020
@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Nov 4, 2020

@ibmdotcom-bot
Copy link
Contributor

ibmdotcom-bot commented Nov 4, 2020

Copy link
Member

@jeffchew jeffchew left a comment

Choose a reason for hiding this comment

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

Looks good! Do we have a place where this can be demoed?

@kennylam
Copy link
Member Author

kennylam commented Nov 5, 2020

Looks good! Do we have a place where this can be demoed?

Not yet, I'll set up a Codesandbox with a mock example.

@jeffchew jeffchew added the Ready to merge Label for the pull requests that are ready to merge label Nov 5, 2020
@kodiakhq kodiakhq bot merged commit 0859a21 into carbon-design-system:master Nov 5, 2020
@kennylam kennylam deleted the feat/custom-masthead-logo branch November 6, 2020 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready to merge Label for the pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants