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

feat(pie-docs): DSW-2268 adds dynamic display of index page cards #1730

Merged
merged 12 commits into from
Sep 10, 2024

Conversation

maledr5
Copy link
Contributor

@maledr5 maledr5 commented Aug 27, 2024

Describe your changes (can list changeset entries if preferable)

Created a shortcode that will display a card for each element in the navigation under a category.
A category is a main navigation item like Components or Foundations.
This shortcode can be used as content for the Category page when we support one. Currently this PR adds support for components, and a new will be implementing Foundations.

Future updates will only need to add the image to assets, or ignore any items that are not supposed to be displayed.

Author Checklist (complete before requesting a review)

  • I have performed a self-review of my code
  • I have added thorough tests where applicable (unit / component / visual)
  • I have reviewed the Docs site preview
  • If there are visual test updates, I have reviewed them properly before approving

Reviewer checklists (complete before approving)

Reviewer 1 - @jamieomaguire

  • I have reviewed the Docs site preview
  • If new links have been added, I have tested them in the preview
  • If there are visual test updates, I have reviewed them

Reviewer 2

  • I have reviewed the Docs site preview
  • If new links have been added, I have tested them in the preview
  • If there are visual test updates, I have reviewed them

@maledr5 maledr5 requested review from a team as code owners August 27, 2024 08:56
Copy link

changeset-bot bot commented Aug 27, 2024

🦋 Changeset detected

Latest commit: 4791ed8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
pie-docs Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pie-design-system-bot
Copy link
Contributor

pie-design-system-bot commented Aug 27, 2024


Fails
🚫 You currently have an unchecked checklist item in your PR description.

Please confirm this check has been carried out – if it's not relevant to your PR, delete this line from the PR checklist.

Generated by 🚫 dangerJS against 4791ed8

@maledr5 maledr5 changed the title feat(pie-docs): DSW-2268 radds dynamic display of index page cards feat(pie-docs): DSW-2268 adds dynamic display of index page cards Aug 27, 2024
@maledr5 maledr5 requested a review from a team as a code owner August 27, 2024 11:25
@maledr5 maledr5 force-pushed the DSW-2268-docs-index-page-nav branch from 9da2828 to c36fc0a Compare August 27, 2024 16:09
throws error when image is not provided and adds support for 320 screens
@maledr5 maledr5 merged commit 392dd5c into DSW-2268-docs-index-page-nav Sep 10, 2024
28 of 31 checks passed
@maledr5 maledr5 deleted the DSW-2268-docs-index-page branch September 10, 2024 07:52
maledr5 added a commit that referenced this pull request Sep 11, 2024
* feat(pie-docs): DSW-2268 refactors navigation to include link to index pages

* feat(pie-docs): DSW-2268 removes resize observer to apply changes on mobile version

* chore(pie-docs): DSW-2268 update changesets file

* fix(pie-docs): DSW-2268 avoids animation when clicking on the same category

* fix(pie-docs): DSW-2268 remove accordion animation for links

* feat(pie-docs): DSW-2268 adds dynamic display of index page cards (#1730)

* feat(pie-docs): DSW-2268 adds dynamic display of index page cards

* feat(pie-docs): DSW-2268 adds hover animation for cards

* feat(pie-docs): DSW-2268 adds images for index page

* test(pie-docs): DSW-2268 update snapshot with latest changes

* docs(pie-docs): DSW-2268 adds missing images and changeset file

* feat(pie-docs): DSW-2268 ingnore items for draft pages

* chore(pie-docs): DSW-2268 update documentation about new feature

* fix(pie-docs): DSW-2268 address PR comments

throws error when image is not provided and adds support for 320 screens

* fix(pie-docs): DSW-2268 fix nav for subpage dropdown items

* fix(pie-docs): DSW-2268 address PR comments

* fix(pie-docs): DSW-2268 adds empty alt attribute for image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants