made logout dropdown look better
Merge request reports
Activity
- Resolved by Daniel Rainer
@danielrainer little update please review again :)
There are three commented-out lines in the e2e tests where I am not sure if they are included intentionally.
I like that the logout button now behaves properly like the other clickable items in the drop-down.
Making the Accept terms checkbox auto-checked is not a good idea in my opinion. The point of such a box is forcing the user to at least look at this area of the screen briefly. If the box is auto-checked, we might as well just put some text like By signing in you agree to our privacy terms.. I cannot comment on the legal requirements here, but if in doubt I would leave the checkbox unchecked as it was before.
assigned to @Michael.Breu and unassigned @danielrainer
-
@Michael.Breu can you give feedback regarding the:
Making the Accept terms checkbox auto-checked is not a good idea in my opinion. The point of such a box is forcing the user to at least look at this area of the screen briefly. If the box is auto-checked, we might as well just put some text like By signing in you agree to our privacy terms.. I cannot comment on the legal requirements here, but if in doubt I would leave the checkbox unchecked as it was before.
I think Daniel might be right and we maybe could just but By signing in you agree to our privacy terms with a link to the terms?
-
added 17 commits
-
1e3f0f10...0126c081 - 15 commits from branch
development - eef4a2d2 - Merge branch 'development' into logout_dropdown_fix
- ca60b8b7 - changed checkbox back to not being autochecked and improved dropdown to...
-
1e3f0f10...0126c081 - 15 commits from branch
mentioned in issue #289 (closed)
added 1 commit
- ba796462 - centralized necessary padding because of fixed navbar
added 2 commits
mentioned in issue #285 (closed)
