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

documentation for select component #31

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

EeveeInGaa
Copy link
Contributor

@EeveeInGaa EeveeInGaa commented May 28, 2024

Add documentation for select component

@coolify-qupaya
Copy link

coolify-qupaya bot commented May 28, 2024

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-07-08 15:28:29 CET

@EeveeInGaa EeveeInGaa requested a review from patrickjahr May 28, 2024 11:05
@EeveeInGaa EeveeInGaa changed the title chore: add documentation documentation for select component Jun 3, 2024
5. Add a loop of the selectedValues signal to the element with `skSelectLabel`.

That's it, your multi selection is ready to go!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Das hier ist die Beschreibung wie die select-Komponente in der Demo-App genutzt wurde.
Jedoch ist das nicht der einzige Weg. Vielleicht ist es hier wichtiger, dem Leser mitzuteilen, wie er den Multiple Mode einschalten kann und wie das Model dann aussieht.
Als ein bool (egal ob signal, property oder observable, etc.) für den multiple Input und dann ändert sich die Value zu einem Array. Wie er dann das skLabel gestaltet bleibt dem Nutzer überlassen. hier kannst du dann gern auf die Demo eingehen, wie es da gemacht wurde. Was meinst du?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok habs geupdated, aber lass und da bitte nochmal drüber reden, ich bin mir noch nicht ganz sicher ob das so verständlich/anwendbar ist.

Copy link
Contributor

@BaggersIO BaggersIO left a comment

Choose a reason for hiding this comment

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

@EeveeInGaa kannst Du bitte noch die Anmerkungen von Patrick durchgehen? :) Dankeschön.

@Markus-Ende
Copy link
Member

@EeveeInGaa wie schaut es denn mit dem PR aus?

@EeveeInGaa
Copy link
Contributor Author

@EeveeInGaa wie schaut es denn mit dem PR aus?

@Markus-Ende hab jetzt die Sachen angepasst, aber muss nochmal mit patrick reden wegen dem multi select.

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

Successfully merging this pull request may close these issues.

4 participants