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

[Hack-together] Sign-in buttons change background color after reloading the page #279

Closed
1 of 2 tasks
richardrigutins opened this issue Aug 16, 2023 · 5 comments · Fixed by #327
Closed
1 of 2 tasks
Assignees
Labels
Hack-together:accepted Accepted after triage and ready for to be done Hack-together

Comments

@richardrigutins
Copy link
Contributor

Are you accessing the project from the GitHub Codespaces?

  • No, I am running the project on my local machine (I will fill in the OS section below)
  • Yes, I am using GitHub Codespaces.

Note: The project is optimized to run on Codespaces. We welcome contributions to enhance it for cross-platform local development, but it's not something we will prioritize right now. Thank you!

Describe the bug
When navigating to the login page of the portal using the login button, all the sign-in buttons appear with their specific background color.
However, after reloading the page, the sign-in buttons all have a white background.
The same thing happens if I navigate directly to the /auth/login.

To Reproduce
Steps to reproduce the behavior:

  1. Run the application with npm run start
  2. Open the portal app
  3. Click on Welcome, Guest on the top right side of the home page, and then Login
  4. Press F5 or reload the page
  5. The sign-in buttons have a white background

Expected behavior
The buttons shouldn't change color when reloading the page.

Screenshots

image

image

OS (please complete the following information):

  • OS: Linux
  • OS Version: codespaces-a87b53 5.15.0-1042-azure
  • npm version: 9.8.1
  • Node.js version: v18.15.0
  • output of azd version: azd version 1.2.0 (commit 99ea7577f0df0df2ba34b677da189fafba18c0f7)

Additional context
None.

@error9098x
Copy link
Contributor

can I take this Issue ?

@juliamuiruri4
Copy link
Collaborator

@richardrigutins I see you have a couple of other issues. Okay if @error9098x works on this?

@richardrigutins
Copy link
Contributor Author

Hi @error9098x, yes you can work on this issue, thank you 😊

@error9098x
Copy link
Contributor

Progress Update I've resolved the issue and am in the process of submitting a pull request.

@error9098x
Copy link
Contributor

Progress Update I've opened the PR #286 please have a look :) @anfibiacreativa

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