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.

Sign up for GitHub

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

Jump to bottom

Bootstrap5 #1434

Merged
merged 39 commits into from
Jul 23, 2024
Merged

Bootstrap5 #1434

merged 39 commits into from
Jul 23, 2024

Conversation

pgiraud
Copy link
Member

@pgiraud pgiraud commented Jul 4, 2024

No description provided.

@pgiraud pgiraud force-pushed the bootstrap5 branch 5 times, most recently from 82cf642 to 45a3da1 Compare July 17, 2024 10:39
@pgiraud pgiraud marked this pull request as ready for review July 17, 2024 10:39
@bersace
Copy link
Member

bersace commented Jul 17, 2024 •
edited
Loading

Dans la dialogue d'édition d'une instance:

image

  • Supprimer la bordure autour de l'identification de l'instance.
  • La croix pour nettoyer l'erreur doit être alignée à droite.

@bersace
Copy link
Member

bersace commented Jul 17, 2024 •
edited
Loading

Dashboard:

image

Ajouter une marge entre la barre latérale et le contenu. C'est valable pour d'autres pages comme statements, maintenance, etc.

@bersace
Copy link
Member

bersace commented Jul 17, 2024

Dashboard:

image

Centrer No alerts.

@bersace
Copy link
Member

bersace commented Jul 17, 2024

Dans Activity:

image

Si je clique sur Yes, terminate, j'ai l'erreur suivante dans la console:

image

Étrange que ça ne remonte pas dans les tests.

bersace
bersace reviewed Jul 17, 2024
View reviewed changes
Copy link
Member

@bersace bersace left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Le découpage des commits est excellent, j'aurai préféré bs5: comme préfixe de message de commit, ça me semble plus cohérent avec le reste. ou ui(bs5): .

@MarionGiusti MarionGiusti force-pushed the bootstrap5 branch 2 times, most recently from 61ee630 to 082f0c1 Compare July 17, 2024 14:03
@pgiraud pgiraud force-pushed the bootstrap5 branch 2 times, most recently from 1bf944b to 64c523f Compare July 17, 2024 14:32
@MarionGiusti MarionGiusti force-pushed the bootstrap5 branch 2 times, most recently from cad615a to fea520d Compare July 18, 2024 10:34
@pgiraud pgiraud requested a review from bersace July 18, 2024 12:15
@bersace
Copy link
Member

bersace commented Jul 22, 2024

Étrange, dans le dashboard, le graphique TPS n'est dessiné qu'au chargement de la page. Puis il se déplace à gauche sans nouvelles données. Il faut actualiser la page pour obtenir un affichage à jour. Aucune erreur dans la console.

image

@bersace
Copy link
Member

bersace commented Jul 22, 2024

Dans Activity:

image

Si je clique sur Yes, terminate, j'ai l'erreur suivante dans la console:

image

Étrange que ça ne remonte pas dans les tests.

J'ai toujours cette erreur.

@bersace
Copy link
Member

bersace commented Jul 22, 2024

Dans /status, les liens sont maintenant soulignés. Peut-on les laisser sans soulignement ?

image

@bersace
Copy link
Member

bersace commented Jul 22, 2024

Dans la dialogue d'édition d'une instance:

image

  • Supprimer la bordure autour de l'identification de l'instance.
  • La croix pour nettoyer l'erreur doit être alignée à droite.

J'ai toujours cet affichage.

@pgiraud
Copy link
Member Author

pgiraud commented Jul 23, 2024

Étrange, dans le dashboard, le graphique TPS n'est dessiné qu'au chargement de la page. Puis il se déplace à gauche sans nouvelles données. Il faut actualiser la page pour obtenir un affichage à jour. Aucune erreur dans la console.

image

cf. #1457

@pgiraud
Copy link
Member Author

pgiraud commented Jul 23, 2024

Dans /status, les liens sont maintenant soulignés. Peut-on les laisser sans soulignement ?

image

Bootstrap a fait le choix de souligner les liens.
https://getbootstrap.com/docs/5.3/migration/#content-reboot-etc

@pgiraud
Copy link
Member Author

pgiraud commented Jul 23, 2024

Dans Activity:
image
Si je clique sur Yes, terminate, j'ai l'erreur suivante dans la console:
image
Étrange que ça ne remonte pas dans les tests.

J'ai toujours cette erreur.

Es-tu bien à jour ?

@bersace
Copy link
Member

bersace commented Jul 23, 2024

Es-tu bien à jour ?

Effectivement, je n'étais pas à jour. J'ai dû rater quelque chose. Désolé !

pgiraud and others added 26 commits July 23, 2024 10:28
@pgiraud
Change css selector in tests for configuration
cd26125 
Instead of relying on CSS class, we select first column.
@pgiraud
ui(bs5): replace badge- color classes
03cd004 
Use bg- instead
@pgiraud
ui(bs5): Update inline forms
7519a27
@pgiraud
ui(bs5): Improve management of custom colors
a1c8a55
@pgiraud
ui(bs5): don't override success theme color
655203a 
For an unknown reason Bootstrap uses a black font when the background is set with this
success color. It may not be dark enough for BS to decide to switch to white font.
@pgiraud
ui(bs5): use required form-label class
cbb5732
@pgiraud
ui(bs5): remove form-group (deprecated)
81383e6 
Use grid system instead
@pgiraud
ui(bs5): Remove input-group-[append|prepend]
0e27d68
@pgiraud
ui(bs5): update form switches
0461f35
@pgiraud
ui(bs5): drop text-justify
c3ef01c
@pgiraud
ui(bs5): replace text-muted (deprecated)
ec646bd
@pgiraud
ui(bs5): Replace navbar-dark
a3ab6af
@pgiraud
ui(bs5): Update modal management
59b1d47
@pgiraud
ui(bs5): add some position-relative
8e4a3cf 
It's required for the position-absolute in children to work properly.
@pgiraud
ui(bs5): Add space for the daterangepicker
e666f9e
@pgiraud
ui(bs5): push close button to the right
591977b
@pgiraud
ui(bs5): Fix metric selector (collapse) opening
31616f0
@pgiraud
ui(bs5): Use a form-switch for boolean settings
7eabd2f
@pgiraud
ui(bs5): add tbody to make border available
27b8fe0
@pgiraud
ui(bs5): revise popover management
71c7b7f 
In maintenance pages, there's no popover apparently, so no need to enable them.
@pgiraud
ui(bs5): Use form-select instead of form-control in pgconf
716f48b
@pgiraud
Remove unused css class in pgconf
2074d71
@pgiraud
ui(bs5): Remove alert-light from Settings dialog components
12db576 
The resulting rendering is different from what we had in BS4.
@pgiraud
ui(bs5): Fix main element width (take gutter into account)
7675972 
This fixes the missing space between sidebar and main element when switching to BS5.
@pgiraud
Center the "No alerts" message in dashboard
4e491bb
@MarionGiusti @pgiraud
Add component SettingSwitch
3fb131a 
Display switch value next to checkbox.
@pgiraud pgiraud merged commit cd268a3 into master Jul 23, 2024
16 checks passed
@pgiraud pgiraud deleted the bootstrap5 branch July 23, 2024 08:37
@pgiraud pgiraud mentioned this pull request Jul 23, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bersace bersace bersace approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pgiraud @bersace @MarionGiusti