Meeds-io / MIPs

The Meeds Improvement Proposal repository
0 stars 0 forks source link

Space members and people page UI/UX enhancement #127

Closed Julien-Dubois-eXo closed 1 month ago

Julien-Dubois-eXo commented 3 months ago

Rationale

We need to unify the space member card with the organizational chart and people directory in the space member tab for UX consistency.

End-users have reported that the card is not useful in some way:

1. Functional Requirements

Top User Stories

In order to unify the UX between cards in organizational chart, the people page, and the space members tab we need to:

Precision: Below, find characteristics of the people card when listing it in the space members app

And finally, some improvements will be done to make the page to harmonize it with the latest components:

Some improvements will be made to make the people page more readable

Remove the "Showing XX results" in the top bar application Display 21 users at first load instead of 20 presently. So we have 7 full lines of 3 in case of max width of the app.

Image

Image

Impacts

Gamification

Notifications

Analytics

Unified Search

2. Technical Requirements

Expected Volume & Performance

Security

Extensibility

User cards components people-list/components/usercard/PeopleCard.vue and its child-components will replace the old people-list/components/PeopleCard.vue and its child components The new card will have two extensibility points for Javascript extensions :

Configurability

Upgradability

Existing Features

Feature Flags

Property Name Default Value Target Audience Functional Behaviour
exo.feature.OrganizationalChart.enabled false All This flag will hide the Organization chart portlet, but won't prevent the creation of the new pages/navigations. If set to true, the Organization chat portlet will be displayed.

Other Non Functional Requirements

3. Impacts

Documentation

Following this MIP, we will add tutorial to help user add action to :

Training

4. Software Architecture

Security

Access

Services & processing

Data and persistence

Clustering

Multitenancy

Integrations

Migration strategy

5. Annexes

Julien-Dubois-eXo commented 3 months ago

Design will comes

Julien-Dubois-eXo commented 3 months ago

@srenault-meeds design added

Julien-Dubois-eXo commented 3 months ago

@ahamdi could you complete the technical part please?

srenault-meeds commented 3 months ago

Go-Func

Julien-Dubois-eXo commented 2 months ago

@srenault-meeds I have merged this MIP https://github.com/orgs/Meeds-io/projects/4/views/1?pane=issue&itemId=56482833 with this one as requested.

srenault-meeds commented 2 months ago

Ok so we close it then

Julien-Dubois-eXo commented 2 months ago

@srenault-meeds it's this one the good one we have to keep.

srenault-meeds commented 2 months ago

Ok I see. I have updated the description to better distinguish with the space members app:

Precision: Below, find characteristics of the people card when listing it in the space members app

srenault-meeds commented 2 months ago

Go-Fonc

margondicco commented 2 months ago

Hello @ahamdi technical part needed

Thank you

Julien-Dubois-eXo commented 2 months ago

@srenault-meeds we are currently working on it. The technical part will be added by @ahamdi quickly.

ahamdi commented 1 month ago

@boubaker Technical spec of MIP-127 updated, feature/mips acceptance updated and PR proposed for the MIP merge on Develop

srenault-meeds commented 1 month ago

Hello

Here are my feedbacks. I let you contact me if you want to discuss it.

I guess the following items have not been implemented:

On Mobile, bottom drawer is too much high, particularly for meeds context. I recommend to make it dynamic with a min-height and a max-height Screenshot_20240506_131332 Screenshot_20240506_130816

ahamdi commented 1 month ago

@srenault-meeds Yes some of the issues you shared are already in the board (issue with disabled users in space members + bottom drawer) They are catched on the board and I am working to fix them all. Are those issues blocker for the merge ?

srenault-meeds commented 1 month ago

Ok no blocker for me. Go

srenault-meeds commented 1 month ago

As precised in the #112 , ok so let's merge it :)