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

[sistent] Add TransferList component to under project sistent #5669

Open
sudhanshutech opened this issue Jul 11, 2024 · 11 comments
Open

[sistent] Add TransferList component to under project sistent #5669

sudhanshutech opened this issue Jul 11, 2024 · 11 comments
Labels
kind/enhancement New feature or request

Comments

@sudhanshutech
Copy link
Member

We have a transferlist component under sistent-repo which we use for environment or connection to assigned transfer from available data to assigned data. Uses can be found here: https://playground.meshery.io/management/workspaces when we click on count section there is a modal open and we assign items .
Add this under https://layer5.io/projects/sistent/components

Screenshots / Mockups

image

Implementation

Acceptance Tests


Contributor Resources and Handbook

The layer5.io website uses Gatsby, React, and GitHub Pages. Site content is found under the master branch.

Join the Layer5 Community by submitting your community member form.

@sudhanshutech sudhanshutech added the kind/enhancement New feature or request label Jul 11, 2024
@rahulharpal1603
Copy link
Contributor

image

Like these two, we have to add the third one named Transfer List?

@NagaHarshita
Copy link

NagaHarshita commented Jul 11, 2024

@sudhanshutech Please assign this issue to me. @rahulharpal1603 In my opinion, guidelines for the TransferList component need to be added similar to the components in screenshot.

I plan to divide it into significant PRs.

  1. Overview
  2. Guidance - explaining each icon significance (>>, <<, <, >)
  3. Code - examples to the common use cases
    I can develop on this idea, subsections with details.

@sudhanshutech
Copy link
Member Author

Sounds like a plan @NagaHarshita . Please try to reuse components as much as you can .

@sudhanshutech
Copy link
Member Author

@rahulharpal1603 you can assist if needed

@sudhanshutech
Copy link
Member Author

@NagaHarshita any luck? Any progress?

@NagaHarshita
Copy link

@NagaHarshita any luck? Any progress?

@sudhanshutech Yes, I have been working on it.
Overview section

  • Brief description transfer list - used to transfer selected items between assign data and available data
  • Multiple select option in checkbox
  • Enabled, Disabled buttons in >>, >, <<, <

Observation - After selected top two items are moved to the assigned data, next two top items are automatically checked. (Is it a design choice ?)

Do I need to add any more subsections in the overview?

@sudhanshutech
Copy link
Member Author

You can raise a pr with do-not-merge label . So we can review your progress

@sudhanshutech
Copy link
Member Author

@NagaHarshita ?

@sudhanshutech
Copy link
Member Author

@NagaHarshita any progress?

@rahulharpal1603
Copy link
Contributor

Since the issue is unassigned, can I start to work on this?
@sudhanshutech

@rahulharpal1603
Copy link
Contributor

@sudhanshutech can I work on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request
Development

No branches or pull requests

3 participants