Skip to content

feat(login): add branding to login redirect page#853

Merged
craicoverflow merged 3 commits intoredhat-developer:mainfrom
craicoverflow:add-branding
Jul 29, 2021
Merged

feat(login): add branding to login redirect page#853
craicoverflow merged 3 commits intoredhat-developer:mainfrom
craicoverflow:add-branding

Conversation

@craicoverflow
Copy link
Contributor

@craicoverflow craicoverflow commented Jul 29, 2021

This PR includes the new product branding into the login web page.

Verification Steps

  1. Run the login page
  2. The login web page should include the new branding on the main body and as a favicon.

image

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation change
  • Other (please specify)

Checklist

  • Documentation added for the feature
  • CI and all relevant tests are passing
  • Code Review completed
  • Verified independently by reviewer

<div class="pf-c-empty-state">
<div class="pf-c-empty-state__content">
<i class="fas fa-key pf-c-empty-state__icon" aria-hidden="true"></i>
<img src="/static/img/logo.png" alt="Logo"/>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will change these to SVG

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes. Inline will be easier than Embed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

What? I will still need to embed them.

Copy link
Collaborator

@wtrocki wtrocki left a comment

Choose a reason for hiding this comment

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

Verified

@craicoverflow craicoverflow merged commit 07ed920 into redhat-developer:main Jul 29, 2021
@craicoverflow craicoverflow deleted the add-branding branch July 29, 2021 15:45
@craicoverflow craicoverflow linked an issue Aug 3, 2021 that may be closed by this pull request
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.

Include the new RHOAS logo in the login redirect page

2 participants