feat(dashboard): person list #1682
Conversation
arnaudambro
temporarily deployed
to
review-auto
GitHub Actions
Inactive
|
This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation. |
arnaudambro
temporarily deployed
to
review-auto
GitHub Actions
Inactive
arnaudambro
temporarily deployed
to
review-auto
GitHub Actions
Inactive
|
Kudos, SonarCloud Quality Gate passed!
|
|
🎉 Deployment for commit e485567 : IngressesDocker images
|
There was a problem hiding this comment.
Je découvre les useMinimumWidth.
Je trouve que le menu sur le côté est bizarre (il occupe un petit tiers de l'écran, a une animation cinématographique et quelques autres trucs).
J'ai un truc très bizarre sur le menu en bas, qui affiche "priorités" pas cliquable.
J'imagine qu'on va avancer par intérations, je pense que c'est bien si j'en fais un peu aussi !
Go pour ça je pense, je ne suis pas trop prêt encore à être critique ni très bon testeur sur le sujet, avançons.
| @@ -181,7 +181,7 @@ const RestrictedRoute = ({ component: Component, _isLoggedIn, ...rest }) => { | |||
| padding: 0 !important; | |||
| overflow: initial; | |||
| */} | |||
| <main className="tw-relative tw-flex tw-grow tw-basis-full tw-flex-col tw-overflow-auto tw-overflow-y-scroll tw-px-2 print:!tw-ml-0 print:tw-h-auto print:tw-max-w-full print:tw-overflow-visible print:tw-p-0 sm:tw-px-12 sm:tw-pt-4 sm:tw-pb-12"> | |||
| <main className="tw-relative tw-flex tw-grow tw-basis-full tw-flex-col tw-overflow-auto tw-overflow-x-hidden tw-overflow-y-scroll tw-px-2 print:!tw-ml-0 print:tw-h-auto print:tw-max-w-full print:tw-overflow-visible print:tw-p-0 sm:tw-px-12 sm:tw-pt-4 sm:tw-pb-12"> | |||
There was a problem hiding this comment.
Je pense qu'on pourra simplifier. Pas hyper fan d'avoir des overflow en dur sur toute la page, d'habitude on n'en a pas besoin, je suis pas sûr de voir ici ? (et pourquoi on est en relative ici).
Mon commentaire est ignorable on peut en reparler plus tard.
# [1.289.0](v1.288.5...v1.289.0) (2023-10-10) ### Features * **dashboard:** person list ([#1682](#1682)) ([342716c](342716c))
|
🎉 This PR is included in version 1.289.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
No description provided.