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

Refactor/enhancments #469

Merged
merged 12 commits into from
Jan 8, 2025
Merged

Refactor/enhancments #469

merged 12 commits into from
Jan 8, 2025

Conversation

ismail9k
Copy link
Owner

@ismail9k ismail9k commented Jan 8, 2025

This pull request includes various updates and improvements to the documentation for the carousel component, including changes to data descriptions, event triggers, method details, component styles, and configuration properties.

Updates to Data Descriptions:

  • Updated the order and descriptions of data properties in docs/api/data.md for better clarity and consistency.

Event Trigger Documentation:

  • Added documentation for the @drag event and updated the descriptions for the @loop and @slide-start events in docs/api/events.md. [1] [2]

Method Details:

  • Removed the outdated slideTo(index: number) method and added the restartCarousel method details in docs/api/methods.md. [1] [2]

Component Styles:

  • Reorganized and updated the CSS variable descriptions for navigation buttons and pagination buttons in docs/components/navigation.md and docs/components/pagination.md. [1] [2]

Configuration Properties:

  • Revised the configuration properties table in docs/config.md to include types and more detailed descriptions.
  • Updated the slots attributes section to reflect the renamed slideSize property.

@ismail9k ismail9k merged commit 0b076ef into master Jan 8, 2025
4 checks passed
@ismail9k ismail9k deleted the refactor/enhancments branch January 8, 2025 11:45
@ismail9k ismail9k restored the refactor/enhancments branch January 15, 2025 14:42
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.

1 participant