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

Unify add/new and delete/remove actions #645

Open
vin0401 opened this issue Oct 24, 2024 · 0 comments
Open

Unify add/new and delete/remove actions #645

vin0401 opened this issue Oct 24, 2024 · 0 comments

Comments

@vin0401
Copy link
Collaborator

vin0401 commented Oct 24, 2024

Design related issue:
https://github.com/pimcore/product-management/issues/643#issue-2569687919

Figma link: https://www.figma.com/design/kTxowT29eCHwNh5NspDGGu/Pimcore-Components?node-id=6690-232596

When do we use each? Add VS New
Add:

New:

Exceptions:

  • add a column in the grid config (more natural word to use)
  • "Add & Edit" have the edit icon; "Edit" also has the edit icon. It is to indicate the user can modify and not only add.

When do we use each? Delete vs Remove/ Cancel/ Discard/ Clear
Delete:

  • Rule - when something gets erased from the system completely/ permanently (fx deleting an asset)
  • Icon - trash bin (icon only and with a label)

Remove/ Cancel/ Discard/ Clear:

  • Rule - still exists in the system but it is not visible on the UI
  • Icon - no icon, only label

Exceptions:

  • "x" icon is only used for closing (mostly on the modals). However "x" as a remove in the grid filters - not to confuse the filters with the config itself. Also in the batch editing following the same design.
  • Object Bricks Field has plus and minus as ways to add and remove bricks - the main icon is edit with the word edit.
  • Many-to-many - delete icon in tables is more common than remove and following the logic that we are not deleting the object itself but the relation, we are still following the ruleset.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant