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

IdP info "dances" behind credentials input dialog #150

Open
pauldekkers opened this issue Dec 4, 2024 · 4 comments
Open

IdP info "dances" behind credentials input dialog #150

pauldekkers opened this issue Dec 4, 2024 · 4 comments
Assignees

Comments

@pauldekkers
Copy link
Member

As demonstrated in this video, the IdP info on the background sometimes "dances" behind the username/password dialog:

ScreenRecording_12-04-2024.13-38-25_1.MP4

Another issue is possibly the fact that first the client asks for a govroam.nl credential but an edu.nl immediately after; this .eap-config is a bit strange and hand-crafted though. Yet, the first mechanism should be usable for iOS if I'm not mistaken (so there's little reason to fallback to another mechanism). It's very very unlikely we encounter this in production. So the dancing is the cosmetic issue here mostly.

@risinggalaxy risinggalaxy self-assigned this Dec 5, 2024
risinggalaxy added a commit that referenced this issue Dec 9, 2024
IdP info "dances" behind credentials input dialog #150
@RianneBoortman
Copy link

Tested on:

  • iPhone 12 Pro Max, iOS 18.1.1
    Version:
  • geteduroam nr 165

❎ test not approved
See attached screen recording. It is a lot better so it seams, but not fully. Please check it again.

ScreenRecording_12-11-2024.13-22-07_1.MP4

@risinggalaxy
Copy link
Collaborator

risinggalaxy commented Dec 12, 2024

@RianneBoortman The “dancing alert” is essentially an iOS thing —it’s a system alert, a layer controlled by iOS itself, and not something we can manipulate. This occurs because the system keyboard changes its frame, which in turn affects the system alert. What was within our control, however, has already been fixed, namely the connect button and the information displayed above it.

@eq-radvile
Copy link

@RianneBoortman see answer above

@RianneBoortman
Copy link

See comment of @risinggalaxy . Unfortunately we cannot do more that we already did
Approving ticket because of this

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

No branches or pull requests

4 participants