Skip to content

Conversation

@nhoening
Copy link
Contributor

@nhoening nhoening commented Dec 18, 2025

Description

Implement two UX improvements on the 2FA form, which we found in a user test.

Closes #1761
See that issue for other suggestions which we are not tackling here.

Look & Feel

Before:

Screenshot from 2025-12-18 15-01-53

After:
image

TODO

  • Fix user message
  • Hide form content which is only used when user has trouble
  • Added changelog item in documentation/changelog.rst

Signed-off-by: Nicolas Höning <nicolas@seita.nl>
@nhoening nhoening requested a review from joshuaunity December 18, 2025 14:37
@nhoening nhoening added the UI label Dec 18, 2025
@nhoening nhoening added this to the 0.31.0 milestone Dec 18, 2025
@nhoening nhoening removed the request for review from joshuaunity December 18, 2025 14:40
@read-the-docs-community
Copy link

read-the-docs-community bot commented Dec 18, 2025

Documentation build overview

📚 flexmeasures | 🛠️ Build #30840008 | 📁 Comparing 7359e6c against latest (970da44)


🔍 Preview build

Show files changed (2 files in total): 📝 2 modified | ➕ 0 added | ➖ 0 deleted
File Status
changelog.html 📝 modified
api/v3_0.html 📝 modified

@nhoening nhoening changed the title Improve initial user message Two UX improvements on the 2FA form Dec 18, 2025
@joshuaunity
Copy link
Contributor

I have seen this page while using the live staging, but on my local, i don't actually come across this page. What can i do to get to see this page?

My local goes straight to dashboard when i login.

@nhoening @Flix6x

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>
Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>
@joshuaunity joshuaunity requested a review from Flix6x December 19, 2025 11:29
@joshuaunity
Copy link
Contributor

Although a different approach was mentioned in the issue, I believe this design would be better. We don't have to create an extra page for the other form(as its a pretty small component), it can still be left here and can be visible on command.

@Flix6x

Copy link
Contributor Author

@nhoening nhoening left a comment

Choose a reason for hiding this comment

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

Looks good but you reverted my work, why?

Also, have you tested if this works on a mobile device? I see you used data-bs-toggle="collapse" so that is a good, stable choice and I expect it would, but let's be sure.

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>
@joshuaunity
Copy link
Contributor

Also, have you tested if this works on a mobile device? I see you used data-bs-toggle="collapse" so that is a good, stable choice and I expect it would, but let's be sure.

I tested this and it works fine

Copy link
Contributor Author

@nhoening nhoening left a comment

Choose a reason for hiding this comment

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

I found something that requires some extra attention ...

joshuaunity and others added 5 commits December 19, 2025 16:25
Co-authored-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: JDev <45713692+joshuaunity@users.noreply.github.com>
Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>
Signed-off-by: JDev <45713692+joshuaunity@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UX suggestions for 2FA from user tests

3 participants