Documentation for kselect

[yeshwanth235]

Description

Issue addressed

#664 (comment)

Addresses #PR# HERE

Before/after screenshots

Before

After

Changelog

• Description: Documentation update for kselect
• Products impact: none
• Addresses: Link(s) to GH issue(s) addressed. Include KDS links as well as links to related issues in a consumer product repository too.
• Components: Kselect
• Breaking: no
• Impacts a11y: no
• Guidance: -

Steps to test

• Check the documentation for slots, emits, methods and props for kselect component. changes should be reflecting

(optional) Implementation notes

At a high level, how did you implement this?

Vue Docgen API generates JSDoc documentation for components.

• Contributor has fully tested the PR manually
• If there are any front-end changes, before/after screenshots are included
• Critical and brittle code paths are covered by unit tests
• The change is described in the changelog section above

Reviewer guidance

• Is the code clean and well-commented?
• Are there tests for this change?
• Are all UI components LTR and RTL compliant (if applicable)?
• Add other things to check for here

Comments

[yeshwanth235]

Hi @AlexVelezLl, please help in reviewing

[MisRob]

Hi @yeshwanth235, thanks! @AllanOXDi and @ozer550 would you please review, and @AlexVelezLl perhaps you could then give it a final look and merge?

[MisRob]

Hi @yeshwanth235, just chiming in to mention @AllanOXDi has review in progress (cc @AlexVelezLl). We will follow-up with you.

[AllanOXDi]

Hi @yeshwanth235, great work on this! I’ve added a few comments suggesting some improvements for your consideration.

[AllanOXDi]

Looks good to me. Thanks @yeshwanth235

[AllanOXDi]

@AlexVelezLl is there anything still missing on this PR - if not we can merge it

[yeshwanth235]

Hi @AllanOXDi, PR approve was revoked by mistake 😅. Please help in approving again.

[AlexVelezLl]

Thanks @yeshwanth235! Just some suggestions to make the wording consistent with the docs of other components 🤗, and some clarifications about some props that werent well described. After this, we can merge this PR :). Thank you!

[AlexVelezLl]

Suggested change

	<!-- @slot Displays the label -->
	<!-- @slot Optional slot as alternative to `label` prop -->

[yeshwanth235]

I apologize for the delay in addressing the PR comments. The comments have now been addressed. Please review at your earliest convenience.

[AlexVelezLl]

No worries @yeshwanth235. Thank you!! 🚀

[AlexVelezLl]

Thank you @yeshwanth235!! Looks good to me! 🚀🚀