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(ui): pageheader custom logo #358

Closed
wants to merge 18 commits into from
Closed

Conversation

franzheidl
Copy link
Contributor

@franzheidl franzheidl commented Sep 13, 2024

feat(ui): Pageheader custom logo

Allow users to pass a custom logo component to the PageHeader and render it.

Changes Made

  • use grid layout in header instead of flexbox as it gives us better control to make sure the logo is scaled correctly while keeping its original aspect ratio
  • add a logo prop that accepts a component to render in a slot
  • render the default Juno logo if true or undefined
  • do not render any logo at all if false or null

Related Issues

Screenshots (if applicable)

Testing Instructions

  1. npm i
  2. npm run test / npm run test PageHeader

Checklist

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • I have made corresponding changes to the documentation (if applicable).
  • My changes generate no new warnings or errors.

Copy link

changeset-bot bot commented Sep 13, 2024

🦋 Changeset detected

Latest commit: 0d29563

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@cloudoperators/juno-ui-components Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

github-actions bot commented Sep 13, 2024

PR Preview Action v1.4.7
🚀 Deployed preview to https://cloudoperators.github.io/juno/pr-preview/pr-358/
on branch gh-pages at 2024-09-16 08:28 UTC

@franzheidl franzheidl changed the title feat(ui): Pageheader custom logo feat(ui): gageheader custom logo Sep 13, 2024
@franzheidl franzheidl changed the title feat(ui): gageheader custom logo feat(ui): pageheader custom logo Sep 13, 2024
Copy link
Contributor

@barsukov barsukov left a comment

Choose a reason for hiding this comment

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

I would suggest to close this PR for now cause @gjaskiewicz works on transition this component to Typescript in current sprint, to avoid future conflicts, if it is urgent then maybe we could add this changes to his PR. The task is #212

@edda
Copy link
Contributor

edda commented Sep 16, 2024

This PR is as good as done and afaik the move to TypeScript for this component hasn't begun yet. We shouldn't wait with merging this for too long as not having this feature prevents us from updating ui-components in some apps (because we changed the default logo). Therefore let's go ahead with this.

ArtieReus and others added 10 commits September 16, 2024 10:38
…one (#349)

* feat(docs): add Juno logos and update README to display the github one

* chore(ci): add missing context for title check
* chore(heureka): fix linting issues

* feat(heureka): uses TopNavigationItem instead of TabNavigationItem

* Create hot-dolphins-know.md

* Revert "chore(heureka): fix linting issues"

This reverts commit b4ecc09.

* feat(heureka): renames all tab names into navEntry

* Create little-onions-obey.md

* Delete .changeset/hot-dolphins-know.md
* chore(greenhouse): adds greenhouse dashboard and apps

* chore(greenhouse): adds greenhouse to allowed scopes

* chore(greenhouse): adjust build scripts

* chore(greenhouse): fixes lint issues in cluster-admin

* chore(greenhouse): add ts-luxon pkg to cluster-admin

* chore(greenhouse): merge main branch

* chore(greenhouse): extend esbuild config to build apps

* chore(greenhouse): add missing entries in package.json and changeset

* chore(ci): add nginx.conf to license ignore path

* chore(ci): update package-lock.json

* fix(build): fixes tests

* chore(example): remove console logs

* fix(greenhouse): add dependent apps to turbo for build

* chore(greenhouse): make packages private in apps
…g filters on login (#354)

* fix(supernova): fixes active filters loading order

* fix(supernova): exports useGlobalsIsURLRead on the right file

* fix(supernova): fixes typo and double check empty var

* fix(supernova): fix typo in naming in methods

* chore(core): migrate greenhouse dashboard (#337)

* chore(greenhouse): adds greenhouse dashboard and apps

* chore(greenhouse): adds greenhouse to allowed scopes

* chore(greenhouse): adjust build scripts

* chore(greenhouse): fixes lint issues in cluster-admin

* chore(greenhouse): add ts-luxon pkg to cluster-admin

* chore(greenhouse): merge main branch

* chore(greenhouse): extend esbuild config to build apps

* chore(greenhouse): add missing entries in package.json and changeset

* chore(ci): add nginx.conf to license ignore path

* chore(ci): update package-lock.json

* fix(build): fixes tests

* chore(example): remove console logs

* fix(greenhouse): add dependent apps to turbo for build

* chore(greenhouse): make packages private in apps

* chore(supernova): add missinge dependecies in useCommunication

* chore(supernova): create a new patch changeset

---------

Co-authored-by: Andreas Pfau <[email protected]>
* chore(greenhouse): add version to package.json files

* chore(ci): add greenhouse to scopes for title check
…ets (#342)

chore(version): update versions with Changesets

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@franzheidl franzheidl marked this pull request as ready for review September 16, 2024 08:46
@franzheidl franzheidl requested a review from a team as a code owner September 16, 2024 08:46
@barsukov barsukov mentioned this pull request Sep 16, 2024
6 tasks
@barsukov
Copy link
Contributor

Close cause of this one:
#363

@barsukov barsukov closed this Sep 16, 2024
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.

6 participants