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

@plone/contents - Enhance Contents View #4756

Open
2 of 6 tasks
sneridagh opened this issue May 12, 2023 · 5 comments · May be fixed by #6025
Open
2 of 6 tasks

@plone/contents - Enhance Contents View #4756

sneridagh opened this issue May 12, 2023 · 5 comments · May be fixed by #6025

Comments

@sneridagh
Copy link
Member

sneridagh commented May 12, 2023

PLIP (Plone Improvement Proposal)

Responsible Persons

Proposer: Víctor Fernández de Alba (@sneridagh)

Seconder:

Abstract

Overhaul, and refactor the contents view. Move it to use own core package @plone/contents, so it's portable to other frameworks or platforms (electron, ionic, etc...) or even use it in Plone ClassicUI

It should address these issues:

Motivation

The contents view is a key element for managing content and should have the best support and UX possible. In addition, we should improve it in order to make it a true mini-app detached from the "site frame", so the usability is better (see Quanta mocks).

image

https://www.figma.com/file/YpaRSKjFcFaAkVYnTSVGuz/Volto-UI-%E2%80%93-Quanta?type=design&node-id=1780-534&t=APRtPBwzIc5APzxL-4

Assumptions

No preconditions.

Proposal & Implementation

Refactor the contents view:

  • Use own core package @plone/contents
  • Use TypeScript
  • Better slicing of components so they are easily shadowable
  • Don't use SemanticUI on it, use react-aria-components
  • Use @plone/client instead of the Volto redux-based request API

Deliverables

Refactored Contents.jsx (and child components) using the Tanstack table library.
Documentation on how to use it outside Volto

Risks

No risks.

Participants

@iFlameing
Put your name here.

@tisto
Copy link
Sponsor Member

tisto commented Sep 10, 2023

@sneridagh what's the status of this? Do we still plan to tackle this in Plone 6.1? Otherwise, I'd move it to 6.2 in our roadmap...

@sneridagh
Copy link
Member Author

@iFlameing you've been working on this, right? What's the status?

@sneridagh
Copy link
Member Author

@tisto nope, I think there will be not enough time for this in 6.1.

@sneridagh sneridagh changed the title Use @tanstack/table for contents view Enhance Contents View Jan 12, 2024
@plone plone deleted a comment from ArmanKukreti Jan 13, 2024
@plone plone deleted a comment from ArmanKukreti Jan 13, 2024
@plone plone deleted a comment from sneridagh Jan 13, 2024
@plone plone deleted a comment from ArmanKukreti Jan 13, 2024
@plone plone deleted a comment from sneridagh Jan 13, 2024
kanhaiya04 added a commit to kanhaiya04/volto that referenced this issue Jan 14, 2024
@sneridagh sneridagh changed the title Enhance Contents View Enhance Contents View - @plone/contents Jan 16, 2024
@sneridagh sneridagh removed this from To do in Plone 6.1 Roadmap Jan 16, 2024
kanhaiya04 added a commit to kanhaiya04/volto that referenced this issue Jan 21, 2024
@tisto
Copy link
Sponsor Member

tisto commented Feb 12, 2024

@sneridagh I moved this to Plone 6.2 in the roadmap, ok with you?

@sneridagh
Copy link
Member Author

@tisto this is not going to happen in 6.1, that's for sure.

My doubt is if we should maintain several projects one for each "desirable" release. Are you aware and you take a look at: https://github.com/orgs/plone/projects/33 ?

I'd go for a milestone rather than use a project for the minor releases. Let's talk about it.

@plone plone locked and limited conversation to collaborators Feb 14, 2024
@sneridagh sneridagh changed the title Enhance Contents View - @plone/contents @plone/contents - Enhance Contents View May 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: No status
Status: Backlog
Status: No status
Development

Successfully merging a pull request may close this issue.

4 participants