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

HTTPS protocol for ibm.com link in the masthead #9567

Closed
2 tasks done
David-Kristek opened this issue Oct 17, 2022 · 0 comments
Closed
2 tasks done

HTTPS protocol for ibm.com link in the masthead #9567

David-Kristek opened this issue Oct 17, 2022 · 0 comments
Assignees
Labels
bug Something isn't working dev Needs some dev work

Comments

@David-Kristek
Copy link

David-Kristek commented Oct 17, 2022

Description

We are working on research.ibm.com, this is requested by our SEO specialist.
The href for the "IBM Logo" on the masthead has HTTP protocol:
Http link in code
At the same time is under useAlternateLogo condition the protocol set to HTTPS:
Https link
The best would be if you would make the protocol configurable. It would be nice to have the ability to use protocol-relative URIs (//www.ibm.com) when an empty protocol is used.

Component(s) impacted

Screenshot 2022-10-17 at 15 34 07

Browser

No response

Carbon for IBM.com version

v1.20.0

Severity

Severity 3 = The problem is visible or noticeable to users but does not impede the usability or functionality. Affects minor functionality, has a workaround.

Application/website

research.ibm.com

Package

@carbon/ibmdotcom-react

CodeSandbox example

https://codesandbox.io/s/github/carbon-design-system/carbon-for-ibm-dotcom/tree/main/packages/react/examples/codesandbox/components/Masthead

Steps to reproduce the issue (if applicable)

No response

Release date (if applicable)

No response

Code of Conduct

@David-Kristek David-Kristek added bug Something isn't working dev Needs some dev work labels Oct 17, 2022
kodiakhq bot pushed a commit that referenced this issue Jan 17, 2023
### Related Ticket(s)

#9567 

### Description

This updates the React masthead logo URL to use `https`.

### Changelog

**Changed**

- React masthead logo URL changed to use `https`

<!-- React and Web Component deploy previews are enabled by default. -->
<!-- To enable additional available deploy previews, apply the following -->
<!-- labels for the corresponding package: -->
<!-- *** "test: e2e": Codesandbox examples and e2e integration tests -->
<!-- *** "package: services": Services -->
<!-- *** "package: utilities": Utilities -->
<!-- *** "RTL": React / Web Components (RTL) -->
<!-- *** "feature flag": React / Web Components (experimental) -->
kennylam added a commit to kennylam/carbon-for-ibm-dotcom that referenced this issue Dec 4, 2023
### Related Ticket(s)

carbon-design-system#9567 

### Description

This updates the React masthead logo URL to use `https`.

### Changelog

**Changed**

- React masthead logo URL changed to use `https`

<!-- React and Web Component deploy previews are enabled by default. -->
<!-- To enable additional available deploy previews, apply the following -->
<!-- labels for the corresponding package: -->
<!-- *** "test: e2e": Codesandbox examples and e2e integration tests -->
<!-- *** "package: services": Services -->
<!-- *** "package: utilities": Utilities -->
<!-- *** "RTL": React / Web Components (RTL) -->
<!-- *** "feature flag": React / Web Components (experimental) -->
kennylam added a commit to kennylam/carbon-for-ibm-dotcom that referenced this issue Jun 11, 2024
### Related Ticket(s)

carbon-design-system#9567 

### Description

This updates the React masthead logo URL to use `https`.

### Changelog

**Changed**

- React masthead logo URL changed to use `https`

<!-- React and Web Component deploy previews are enabled by default. -->
<!-- To enable additional available deploy previews, apply the following -->
<!-- labels for the corresponding package: -->
<!-- *** "test: e2e": Codesandbox examples and e2e integration tests -->
<!-- *** "package: services": Services -->
<!-- *** "package: utilities": Utilities -->
<!-- *** "RTL": React / Web Components (RTL) -->
<!-- *** "feature flag": React / Web Components (experimental) -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dev Needs some dev work
Projects
None yet
Development

No branches or pull requests

7 participants