Skip to content

RHIDP-7614: Displaying the preferred username and Company Logo #1254

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

jmagak
Copy link
Member

@jmagak jmagak commented Jul 2, 2025

IMPORTANT: Do Not Merge - To be merged by Docs Team Only

Version(s): 1.7, main

Issue: RHIDP-7614

Preview:

7.3. Displaying company logo in the global header
7.4. Displaying the preferred username in the profile dropdown

@rhdh-bot
Copy link
Collaborator

rhdh-bot commented Jul 2, 2025

@debsmita1
Copy link
Member

@its-mitesh-kumar to review the compant logo

@debsmita1
Copy link
Member

profile display name LGTM !

@jmagak jmagak force-pushed the RHIDP-7614-Global-Header-Add-Company-Logo-and-User-profile-icon-support branch from 5997a85 to 1a6ae62 Compare July 9, 2025 07:58
@jmagak jmagak force-pushed the RHIDP-7614-Global-Header-Add-Company-Logo-and-User-profile-icon-support branch from 1a6ae62 to 2bc3691 Compare August 4, 2025 10:23
@its-mitesh-kumar
Copy link
Member

@jmagak Also we need to add some documentation on how user can now enable the logo in sidebar.


The `CompanyLogo` supports strict display constraints to ensure visual consistency:

* Maximum dimensions: 150px wide, 40px high.
Copy link
Member

Choose a reason for hiding this comment

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

Now width and height can be provided through the configuration.

Copy link
Member

Choose a reason for hiding this comment

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

* The logo must retain its original proportions.
* The component layout remains fixed and cannot be moved from its mount point.
* Supports the following props:
** `logo`: Encoded logo image.
Copy link
Member

Choose a reason for hiding this comment

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

Now theme-specific logo can be provided through configuration.
redhat-developer/rhdh#3023


The `CompanyLogo` supports strict display constraints to ensure visual consistency:

* Maximum dimensions: 150px wide, 40px high.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
* Maximum dimensions: 150px wide, 40px high.
* Default dimensions:
- Width: 150px
- Maximum Height: 40px

* The default maximum dimensions: `150px` wide, `40px` high.
* The logo must retain its original proportions using `object-fit: contain`.
* The component keeps its layout fixed and you cannot move it from its mount point.
* `CompanyLogo` automatically scales the image down if dimensions exceed allowed size.
Copy link
Member

Choose a reason for hiding this comment

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

Instead of this maybe below points make more sense

  • The image is never cropped or distorted.
  • If the configured width would result in a height greater than the allowed maximum (default: 40px), it is automatically scaled down.
  • This means: in some cases, changing only the width may not visibly affect the logo unless height is also adjusted.

dark: 'data:image/svg+xml;base64,...'
----
+
`CompanyLogo` uses the following configuration elements to control fallback and sizing behavior:
Copy link
Member

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants