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

feat: add options button menu in infinite table #24

Merged
merged 5 commits into from
Oct 4, 2024

Conversation

mguellsegarra
Copy link
Collaborator

@ecarreras
Copy link
Member

Ara he vist però que hi estem afegint la dependència d'ant i és un component que estem utilitzant a altres llocs amb material-ui justament perquè no tenia dependències d'altres frameworks

@ecarreras
Copy link
Member

És podria mirar això? https://github.com/react-component/dropdown crec que són els mateixos d'antd que estan intentant fer uns components base sense estilar

@mguellsegarra
Copy link
Collaborator Author

Ara he vist però que hi estem afegint la dependència d'ant i és un component que estem utilitzant a altres llocs amb material-ui justament perquè no tenia dependències d'altres frameworks

Ostres, no hi havia caigut, sorry. Ho implementaré sense fer servir ant, la lib aquesta és genial.

@mguellsegarra
Copy link
Collaborator Author

He ajustat que ara es faci servir la llibreria rc-dropdown, així no arrossguem tot l'ant en projectes que no el facin servir. A part, per les dues icones, les he afegit en format SVG, amb el que tampoc arrosseguem els ant-icons. Feu-hi una ullada a v3.0.0-alpha.159 :)
@ecarreras

@mguellsegarra mguellsegarra merged commit c8403f4 into v2 Oct 4, 2024
1 check passed
@mguellsegarra mguellsegarra deleted the feat/add-options-button-in-table branch October 4, 2024 06:47
github-actions bot pushed a commit that referenced this pull request Oct 4, 2024
# [1.8.0](v1.7.0...v1.8.0) (2024-10-04)

### Features

* add options button menu in infinite table ([#24](#24)) ([c8403f4](c8403f4))
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.

2 participants