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

Update login page #2736

Merged
merged 14 commits into from
Feb 24, 2021
Merged

Update login page #2736

merged 14 commits into from
Feb 24, 2021

Conversation

bjoernricks
Copy link
Member

@bjoernricks bjoernricks commented Feb 19, 2021

What:

Update login page to use same terms and new logo.

Why:

Every GSM gets an newly designed logo therefore GSA gets a fitting logo too. To have a common login page for GSA and the cloud product we should use the same terms too.

How:

Checklist:

Adjust size of the logo at the login page to be more visible and to
better fit into the page. The old logo was smaller. The new one is more
a square.
Use the useGmp hook insted of the withGmp HOC. Using hooks is
much easier to understand then the HOCs.
Pass also all other params for the react class component to the parent
class.
Without the width the panel shrinks a lot and gets to small.
Render the panel as a flexbox.
Add a powered by Greenbone logo to the bottom of the login form.
Open Greenbone website when clicking on the powered by Greenbone logo.
Use same wording as our cloud frontent.
Use hooks for easier understanding of the LoginForm. It is a very basic
form component and is more readable a as function component using hooks.
@bjoernricks bjoernricks changed the base branch from master to gsa-21.04 February 24, 2021 08:28
@bjoernricks bjoernricks marked this pull request as ready for review February 24, 2021 08:30
@bjoernricks bjoernricks requested a review from a team February 24, 2021 08:30
gsa/src/web/components/img/powered.js Outdated Show resolved Hide resolved
gsa/src/web/components/img/powered.js Outdated Show resolved Hide resolved
gsa/src/web/pages/login/loginform.js Outdated Show resolved Hide resolved
gsa/src/web/pages/login/loginform.js Outdated Show resolved Hide resolved
y0urself
y0urself previously approved these changes Feb 24, 2021
The height of the product image has changed with the new image.
Use a more expressive name for the component.
@bjoernricks bjoernricks merged commit 93269c3 into greenbone:gsa-21.04 Feb 24, 2021
@bjoernricks bjoernricks deleted the update-login-page branch February 24, 2021 09:41
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.

None yet

2 participants