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

[BUGFIX] Supprimer l'overflow mis par défaut sur les table dans PixOrga #11584

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

xav-car
Copy link
Contributor

@xav-car xav-car commented Mar 6, 2025

🥞 Problème

en modile, c'est cassé. un méchant copy paste c'est glissé dans le css

🥓 Proposition

Supprimer la ligne que nous ne souhaitons pas

🧃 Remarques

RAS

😋 Pour tester

Aller sur un tableau en mode mobile et vérifier que l'on puisse scroll.

@xav-car xav-car added 👀 Func Review Needed Need PO validation for this functionally team-prescription labels Mar 6, 2025
@xav-car xav-car self-assigned this Mar 6, 2025
@xav-car xav-car marked this pull request as ready for review March 6, 2025 12:41
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@xav-car xav-car force-pushed the fix-css-table-orga branch from 614dce9 to 48a509e Compare March 6, 2025 15:14
@xav-car xav-car force-pushed the fix-css-table-orga branch from 48a509e to ac6af2d Compare March 7, 2025 10:32
@@ -1,6 +1,5 @@
.table {
margin-bottom: var(--pix-spacing-4x);
overflow: unset;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

par contre il fait des trucs chelou sur les drop down d'action. je pense que c'est elle qui sont à revoir. et non le PixTable

( genre dans le tableau membre c'est cocasse)

@Alexandre-Monney
Copy link
Contributor

Euuuuuuuh
image

@xav-car
Copy link
Contributor Author

xav-car commented Mar 7, 2025

Euuuuuuuh image

Tu bluffes martoni
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👀 Func Review Needed Need PO validation for this functionally team-prescription
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants