Skip to content

Table component implementation #71

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
Merged
merged 1 commit into from
Apr 18, 2020
Merged

Table component implementation #71

merged 1 commit into from
Apr 18, 2020

Conversation

bexsoft
Copy link
Collaborator

@bexsoft bexsoft commented Apr 18, 2020

What does this do?

Implementation of a new table wrapper component that will be used across the application.

NOTE: This PR only includes the table in Users page, remaining sections will be updated accordingly in the next PR

How to Test

  1. Go into Users page
  2. Check that the styles correspond as much as possible as the design in XD
  3. Check that functionality for Users is not broken (Add / Edit / Delete / Bulk Users)

Table component implementation
3fac2a2 
Implementation of a new table wrapper component that will be used across the application.
@bexsoft bexsoft self-assigned this Apr 18, 2020
@dvaldivia dvaldivia merged commit 0bcf88e into minio:master Apr 18, 2020
@bexsoft bexsoft deleted the table-rebrand branch May 8, 2020 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dvaldivia dvaldivia dvaldivia approved these changes

@cesnietor cesnietor Awaiting requested review from cesnietor

+1 more reviewer

@Alevsk Alevsk Alevsk approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@bexsoft bexsoft

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bexsoft @Alevsk @dvaldivia