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

Redesign: admin #11144

Merged
merged 723 commits into from Sep 22, 2023
Merged

Redesign: admin #11144

merged 723 commits into from Sep 22, 2023

Conversation

jorgeatgu
Copy link
Contributor

@jorgeatgu jorgeatgu commented Jul 7, 2023

🎩 What? Why?

This PR implements the redesign in admin

📌 Related Issues

Testing

The URL is a bit different for the admin, but the credentials and db are the same.

Admin staging: https://decidim-redesign-admin.populate.tools/admin/
Participatory processes: https://decidim-redesign-admin.populate.tools/admin/participatory_processes
Settings: https://decidim-redesign-admin.populate.tools/admin/organization/edit

♥️ Thank you!

Feedback @alecslupu:

@jorgeatgu jorgeatgu added the contract: redesign Barcelona City Council contract label Jul 7, 2023
@furilo furilo added this to Integrating / Front in PX Redesign Jul 10, 2023
Base automatically changed from feature/redesign to develop July 10, 2023 09:13
@furilo furilo moved this from Integrating / Front to Populate review in PX Redesign Jul 13, 2023
@jorgeatgu jorgeatgu marked this pull request as ready for review July 27, 2023 08:41
@ferblape ferblape requested a review from a team July 27, 2023 12:33
@ferblape
Copy link
Contributor

@decidim/product notice the url is different, but all the credentials are the same (we duplicated the staging database 2 weeks ago).

@entantoencuanto entantoencuanto force-pushed the feature/redesign-admin branch 2 times, most recently from 9a36665 to 3f2197b Compare August 3, 2023 21:27
@andreslucena andreslucena added this to the 0.28.0 milestone Aug 23, 2023
@alecslupu
Copy link
Contributor

This PR has become way to big, considering what we previously agreed, however browsing through it i could find some structural issues, or at least this is my first impression.

1a. In the process, we have 2 tabs one for Processes and one for Process groups, action for each one of the sections being collapsed in right "..." menu (including Process types, which can reside in its own tab entry)
1b. In the assemblies, we have no tabs. The assembly types are missing completely (even though the page is redesigned)
1c. In the initiatives, we have no tabs. The "..." is hiding "initiatives", "Initiative types" and "Settings"

  1. Global moderation there should be 2 tabs ( there are 2 entries now hidden in "..." menu)

My first impression after seeing the admin interface, is that some of the elements are not placed in a visible space, making hard for a new comer to easily understand the Decidim.

Here is a list of things that i had to do:

  1. Open the admin
  2. click Processes
  3. Click a participatory space (At this point i already see the components in the right with published state)
  4. From the "..." Menu i could see Components
  5. Clicking there I could see add component
  6. On where i could add a new Component.

On the Participatory processes, i could see that navigating to:

  • The Process i had a right toolbar containing components,
  • the "About process" i have a left sidebar

Some other unstructured feedback:

  1. The width of the admin is not full screen: ( the .title-bar has a width of about 1800px, but the container is just 1500px, resulting a 250 - 300px bar in the right)
  2. Is we are using the Tabbing concept, then the process Attachments could be added as a single menu entry that would then lead to a tabbed view
  3. In the Process before "Actividad reciente" we could add some kind of cell lock that would allow us to add some common activities "Moderate content", "Add a new component" etc
  4. There is no visual confirmation on permissions being applied (in 0.27 we would have the key turning yellow if there were permissions assigned)
  5. Some of the forms, do not have the desired look and feel even though they have been touched. ( edit meeting for example )
  6. The Language drop down + validation is too packed together

Attachments

image
image

Meeting form

Meeting edit form

Length + Language

image

@Crashillo
Copy link
Member

Finally, specs are passing and all comments has been fixed or addressed 🎉

Copy link
Contributor

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

There is a lot of work added to this PR, and during the Review we have found several design / code issues regarding the implementation. As a result, during our common meetings, we have decided to split the work in sub tasks that can be done at a later stage, by various individuals so that we respect the time and budgets constraints:

Also, due to setup difficulties, the elections / votings module has not been properly reviewed.
Overall review progress at approval time is 95.4% (782 / 819 files viewed)

The list of tasks:

Types of issues that we will touch at a later stage are mostly design & product related:

  • Extra parameters passed to form objects ( titles )
  • Duplicate page titles ( as part of the page & part of the card )
  • Wrongly placed buttons ( ex Buttons placed under the title bar in 2 files )
  • Missing styles on forms
  • Missing or Unneeded sidebars
  • Extra parameters passed to pagination

As result of all of the above this PR is

APPROVED

@alecslupu alecslupu dismissed carolromero’s stale review September 22, 2023 08:50

Dismissing the review. As agreed, will be tackled by maintainers or redesign team, at a later date, starting from the example provided by redesign team.

@alecslupu alecslupu merged commit cc0de0b into develop Sep 22, 2023
76 checks passed
@alecslupu alecslupu deleted the feature/redesign-admin branch September 22, 2023 08:54
@alecslupu alecslupu mentioned this pull request Oct 3, 2023
@andreslucena andreslucena added the type: feature PRs or issues that implement a new feature label Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contract: redesign Barcelona City Council contract type: feature PRs or issues that implement a new feature
Projects
Status: Done
PX Redesign
Populate review
7 participants