Skip to content

Authentification : container trop large#2210

Merged
arnaudlevy merged 13 commits intomainfrom
fix-two-factor-authentication-layout
Oct 1, 2024
Merged

Authentification : container trop large#2210
arnaudlevy merged 13 commits intomainfrom
fix-two-factor-authentication-layout

Conversation

@Olivia206
Copy link
Copy Markdown
Contributor

@Olivia206 Olivia206 commented Sep 23, 2024

Capture d’écran 2024-09-23 à 17 02 52

Intrus dans la PR : fix #2154

arnaudlevy
arnaudlevy previously approved these changes Sep 23, 2024
Copy link
Copy Markdown
Member

@arnaudlevy arnaudlevy left a comment

Choose a reason for hiding this comment

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

J'avais fait un système pour gérer les écrans étroits dans le layout, mais je sais pas si on utilise le même layout que pour l'admin. Si oui, ce serait bien de le réutiliser ?

Comment thread app/views/devise/two_factor_authentication/show.html.erb Outdated
@Olivia206
Copy link
Copy Markdown
Contributor Author

@arnaudlevy ce n'est pas le même layout mais je l'ai intégré en me disant que c'est mieux de conserver la même logique (?)

@arnaudlevy
Copy link
Copy Markdown
Member

J'ai repris et ajouté des minifix ailleurs

@arnaudlevy
Copy link
Copy Markdown
Member

@Olivia206 maintenant c'est limite trop petit, et le logo a vraiment bcp de marge autour, non ?

Capture d’écran 2024-09-30 à 19 17 28

@arnaudlevy arnaudlevy merged commit 4a3012c into main Oct 1, 2024
@arnaudlevy arnaudlevy deleted the fix-two-factor-authentication-layout branch October 1, 2024 08:55
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

Successfully merging this pull request may close these issues.

Supprimer le caption des datatables

3 participants