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

Admin Redesign: Redesiging the People Management Screen #1550

Closed
aashimawadhwa opened this issue Feb 6, 2024 · 13 comments
Closed

Admin Redesign: Redesiging the People Management Screen #1550

aashimawadhwa opened this issue Feb 6, 2024 · 13 comments
Assignees
Labels
feature request no-issue-activity No issue activity ui/ux issue related and being worked with the figma file of the Admin UI

Comments

@aashimawadhwa
Copy link
Member

Is your feature request related to a problem? Please describe.
we want consistent design for people management

Describe the solution you'd like
redesign the people management as of the designs in the figma file

Additional context
Screenshot 2024-02-06 at 10 09 00 PM

link to the figma file: https://www.figma.com/file/WksD4uIa5sQJMIk3pR4l0c/Talawa-Design-System-(Official)?type=design&node-id=307-212&mode=design&t=jw1Q8PosWMSmrFRM-0

Potential internship candidates
Please read this if you are planning to apply for a Palisadoes Foundation internship PalisadoesFoundation/talawa#359

@github-actions github-actions bot added ui/ux issue related and being worked with the figma file of the Admin UI unapproved labels Feb 6, 2024
@duplixx
Copy link
Member

duplixx commented Feb 6, 2024

@palisadoes can i work on this

@crocmons
Copy link

crocmons commented Feb 6, 2024

@palisadoes can I work on it?

@MSVaibhav4141
Copy link

I would like to work on this.

@duplixx
Copy link
Member

duplixx commented Feb 7, 2024

@aashimawadhwa these radio-buttons were not planned in designs where should i put these

image

@duplixx
Copy link
Member

duplixx commented Feb 7, 2024

@aashimawadhwa @palisadoes, I'd like to suggest a UI enhancement: transitioning from tables to cards. Tables aren't always the optimal way to present data, and cards offer a more visually appealing and user-friendly layout, especially considering the recent changes to the UI. Moreover, if we revert to a table format, we'll need to ensure that actions are appropriately placed. However, in the Figma UI, there isn't any provision for selecting fields to perform a single action across multiple items. Hence, sticking with the cards layout seems like the best option since it's already aligned with our current UI. Let me know your thoughts on this suggestion!

here are my current changes
(little tweaks are still left here)

WhatsApp Image 2024-02-07 at 13 33 36_30378182

@varshith257
Copy link
Member

varshith257 commented Feb 10, 2024

@aashimawadhwa @Cioppolo14 I think it is a duplicate issue of #1451

@duplixx
Copy link
Member

duplixx commented Feb 10, 2024

@varshith257 I think there are multiple steps we need to take care of #1451:

1.Redesign Orgpeople and remove the actions button.
2.The user profile CRUD page should match that of the Figma design with the following modifications.

I have already worked on first part . We can divide it into two steps and you can work on 2nd part

@palisadoes
Copy link
Contributor

@varshith257 I think there are multiple steps we need to take care of #1451:

1.Redesign Orgpeople and remove the actions button. 2.The user profile CRUD page should match that of the Figma design with the following modifications.

I have already worked on first part . We can divide it into two steps and you can work on 2nd part

One issue per assignee. Create a separate issue if you feel 2 PRs are required, one each per assignee.

@varshith257
Copy link
Member

varshith257 commented Feb 10, 2024

@palisadoes Already #1451 consists of this issue. Your suggestion required, whether it is a duplicate issue of it or separate issue

@varshith257
Copy link
Member

I think it's a duplicate issue to it. But @duplixx already completed half of the task as in #1451 of redesigning the people management screen. I'll take over from here and ensure that their efforts are not in vain. Together, we can build on the progress

@duplixx
Copy link
Member

duplixx commented Feb 10, 2024

@varshith257 yeah sure !!
After finishing second point raise the PR naming like User Profile Figma update

I think it's a duplicate issue to it. But @duplixx already completed half of the task as in #1451 of redesigning the people management screen. I'll take over from here and ensure that their efforts are not in vain. Together, we can build on the progress

@Cioppolo14
Copy link

@duplixx @varshith257 Please mention this plan in your PR description so we can keep track of it all. Thank you for working together so both of you can contribute.

Copy link

This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request no-issue-activity No issue activity ui/ux issue related and being worked with the figma file of the Admin UI
Projects
None yet
Development

No branches or pull requests

7 participants