Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

DECANTER-193: Start of accordion and development environment. #642

Merged
merged 8 commits into from
May 27, 2020

Conversation

sherakama
Copy link
Member

NOT READY

Summary

  • New Accordion Component
  • A development environment to replace the KSS generator

Needed By (Date)

  • When does this need to be merged by?

Urgency

  • How critical is this PR?

Steps to Test

  1. Do this
  2. Then this
  3. Then this

Affected Projects or Products

  • Does this PR impact any particular projects, products, or modules?

Associated Issues and/or People

  • JIRA ticket
  • Other PRs
  • Any other contextual information that might be helpful (e.g., description of a bug that this PR fixes, new functionality that it adds, etc.)
  • Anyone who should be notified? (@mention them here)

See Also

@sherakama sherakama temporarily deployed to Tugboat March 11, 2020 06:14 Destroyed
@sherakama
Copy link
Member Author

@yvonnetangsu FYI. Not much progress on the accordion but I did manage to scaffold a small development environment to help with building out and testing stuff.

@sherakama sherakama temporarily deployed to Tugboat March 11, 2020 06:41 Destroyed
@sherakama sherakama temporarily deployed to Tugboat March 11, 2020 07:01 Destroyed
@sherakama sherakama temporarily deployed to Tugboat March 15, 2020 00:20 Destroyed
@sherakama sherakama temporarily deployed to Tugboat March 22, 2020 00:23 Destroyed
@sherakama sherakama temporarily deployed to Tugboat March 29, 2020 00:28 Destroyed
#643)

* WIP - added a bit more accordion twig/json/scss stuff and dev page bits expanded upon Shea's decanter-193 branch; some things are there temporarily

* Chipping away at the accordion scss

* beginning js and style fixup

* More aria js and scss

* Change +/- icon when title is clicked

* Try a ul structure for the accordion itself which is more sematic but allows for more flexibility compared to dl

* style chipping

* Add CTA button and title

* Add expand/collapse all buttons and functionality

Co-authored-by: Yvonne Tang <ytang2@stanford.edu>
@sherakama sherakama temporarily deployed to Tugboat March 30, 2020 22:11 Destroyed
@sherakama sherakama linked an issue Apr 2, 2020 that may be closed by this pull request
@sherakama sherakama removed a link to an issue Apr 2, 2020
@buttonwillowsix buttonwillowsix linked an issue Apr 9, 2020 that may be closed by this pull request
@sherakama sherakama temporarily deployed to Tugboat April 23, 2020 18:05 Destroyed
@sherakama sherakama requested a review from jenbreese April 23, 2020 18:06
@yvonnetangsu
Copy link
Member

I know this still need some work but wondering if it's ok to merge as is? It would be nice to get that new development environment @sherakama set up into master 😄

@sherakama
Copy link
Member Author

Let's mark it as experimental and merge it in.

Copy link
Member Author

@sherakama sherakama left a comment

Choose a reason for hiding this comment

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

GTG

@sherakama sherakama merged commit 62ce577 into master May 27, 2020
@sherakama sherakama deleted the decanter-193 branch May 27, 2020 17:47
@yvonnetangsu
Copy link
Member

Let's mark it as experimental and merge it in.

Cool. Thanks!!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create Accordion component
3 participants