Skip to content

2FA icon is misaligned #3431

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

Open
yubiuser opened this issue May 3, 2025 · 6 comments
Open

2FA icon is misaligned #3431

yubiuser opened this issue May 3, 2025 · 6 comments

Comments

@yubiuser
Copy link
Member

yubiuser commented May 3, 2025

When enabling 2FA, the icon in the modal is off

Image

@XhmikosR
Copy link
Contributor

XhmikosR commented May 4, 2025

Yeah, that markup is very wrong

Look for other cases with input-group-addon and follow the same markup. Also, drop form-control-feedback doesn't belong to the icon.

@XhmikosR
Copy link
Contributor

XhmikosR commented May 7, 2025

I'll take a stab at it after my refactoring patches are upstreamed. We need to clear some fields when the modal closes.

@rdwebdesign
Copy link
Member

Looks like this is an issue of CSS order.

I think if we load fontawesome CSS before bootstrap the issue will be fixed.

@XhmikosR
Copy link
Contributor

XhmikosR commented May 9, 2025

Or rather we should use the proper Bootstrap 3 markup instead...

@rdwebdesign
Copy link
Member

Yeah... my suggestion was made by just looking at the selectors, but looking at the HTML markup, I understand your point.

This markup was created more than 2 years ago (a long time before v6 release) when we were porting the PHP code to Lua. At that time no one payed attention to details and we never tried to fix it.

@XhmikosR
Copy link
Contributor

XhmikosR commented May 9, 2025

Yeah, no worries, it's just that there are a few such cases and it just doesn't make sense to not follow the official markup when possible unless we can achieve what we want.

Anyway, this code needs further tweaking later apart from the markup, so that the modal content is cleared when the modal is closed since when it's open we request new data and until the new data comes up, the old is shown.

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

3 participants