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

Authentication on the Home page, user name is not visible due to the wrong color; should be the same of List and Map pages #847

Open
mbarto opened this issue Feb 16, 2021 · 1 comment

Comments

@mbarto
Copy link
Contributor

mbarto commented Feb 16, 2021

Issue from document: "Segnalazioni_DECSIRAv2_v2.docx" sent by client on 12/02/2021, point 4.

https://drive.google.com/file/d/1Kb_DqO675U5sLbWpe2J4cP9BSEHk2XOl/view?usp=sharing

image

image

Moreover, if the user is authenticated, the icon is not shown in green.

@mbarto mbarto changed the title Autenticazione su Home non si vede il nome dell’utente per via del colore vedi invece come in pagina Lista e Mappa Authentication on the Home page, user name is not visible due to the wrong color; should be the same of List and Map pages Feb 16, 2021
dsuren1 added a commit to dsuren1/csi-sira that referenced this issue Feb 26, 2021
(cherry picked from commit 46fd05e73be2306bdf9e5f18946c42a1ea6ea893)
@dsuren1 dsuren1 mentioned this issue Feb 26, 2021
6 tasks
mbarto pushed a commit that referenced this issue Mar 1, 2021
(cherry picked from commit 46fd05e73be2306bdf9e5f18946c42a1ea6ea893)
@mbarto
Copy link
Contributor Author

mbarto commented Mar 2, 2021

@dsuren1 I had some feedbacks from the client on this:

The color of the icon is still not correct:

The icon is green both for logged and not logged users, and becomes black on mouse over.
image
image

The correct behaviour is this:

  • always black (no color change onMouseOver)
  • the background is green for logged users and becomes a lighter green on mouse over

Not logged user:
image

Logged user:
image

Logged user on mouse over:
image

IMPORTANT:
Logout has no effect!

dsuren1 added a commit to dsuren1/csi-sira that referenced this issue Mar 3, 2021
(cherry picked from commit 4579663cf4884b4ca497a80d1ed1178c1e986597)
@dsuren1 dsuren1 mentioned this issue Mar 3, 2021
6 tasks
mbarto pushed a commit that referenced this issue Mar 3, 2021
(cherry picked from commit 4579663cf4884b4ca497a80d1ed1178c1e986597)
mbarto added a commit that referenced this issue Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants