Skip to content

Conversation

@starryeyez024
Copy link
Member

@starryeyez024 starryeyez024 commented Dec 7, 2018

This component toggles between the tabs and the accordion components depending on the space available. If the space is smaller than 768px, the accordion is rendered; otherwise, the tabs are rendered.

@castastrophe castastrophe self-assigned this Dec 14, 2018
@castastrophe castastrophe self-requested a review December 14, 2018 16:53
@castastrophe castastrophe changed the title WIP Us188751 hide show component US188751 Hide-show component Dec 14, 2018
@starryeyez024
Copy link
Member Author

@chrisdo1 @castastrophe I'm getting this error in storybook - are you?

Cannot find module "lorem-ipsum"
    
      Error: Cannot find module "lorem-ipsum"
    at webpackMissingModule (http://localhost:9001/static/preview.bundle.js:35670:77)
    at Object. (http://localhost:9001/static/preview.bundle.js:35670:165)
    at Object. (http://localhost:9001/static/preview.bundle.js:35727:30)
    at __webpack_require__ (http://localhost:9001/static/preview.bundle.js:679:30)
    at fn (http://localhost:9001/static/preview.bundle.js:89:20)
    at webpackContext (http://localhost:9001/static/preview.bundle.js:32521:9)
    at http://localhost:9001/static/preview.bundle.js:30133:12
    at Array.forEach ()
    at loadStories (http://localhost:9001/static/preview.bundle.js:30128:14)
    at ConfigApi._renderMain (http://localhost:9001/static/preview.bundle.js:31107:20)
    
  
ERROR in ./elements/pfe-hide-show/pfe-hide-show.story.js
Module not found: Error: Can't resolve 'lorem-ipsum' in '/Users/ktotten/Sites/rhelements/elements/pfe-hide-show'
ERROR in ./elements/pfe-tabs/pfe-tabs.story.js
Module not found: Error: Can't resolve 'lorem-ipsum' in '/Users/ktotten/Sites/rhelements/elements/pfe-tabs'

@chrisdo1
Copy link
Contributor

@chrisdo1 @castastrophe I'm getting this error in storybook - are you?

Cannot find module "lorem-ipsum"
    
      Error: Cannot find module "lorem-ipsum"
    at webpackMissingModule (http://localhost:9001/static/preview.bundle.js:35670:77)
    at Object. (http://localhost:9001/static/preview.bundle.js:35670:165)
    at Object. (http://localhost:9001/static/preview.bundle.js:35727:30)
    at __webpack_require__ (http://localhost:9001/static/preview.bundle.js:679:30)
    at fn (http://localhost:9001/static/preview.bundle.js:89:20)
    at webpackContext (http://localhost:9001/static/preview.bundle.js:32521:9)
    at http://localhost:9001/static/preview.bundle.js:30133:12
    at Array.forEach ()
    at loadStories (http://localhost:9001/static/preview.bundle.js:30128:14)
    at ConfigApi._renderMain (http://localhost:9001/static/preview.bundle.js:31107:20)
    
  
ERROR in ./elements/pfe-hide-show/pfe-hide-show.story.js
Module not found: Error: Can't resolve 'lorem-ipsum' in '/Users/ktotten/Sites/rhelements/elements/pfe-hide-show'
ERROR in ./elements/pfe-tabs/pfe-tabs.story.js
Module not found: Error: Can't resolve 'lorem-ipsum' in '/Users/ktotten/Sites/rhelements/elements/pfe-tabs'

@starryeyez024 Yep, I got the same error. Here's what I did:

  1. deleted node modules in the pfe root
  2. ran npm install again
  3. you'll probably see an error like this one: 404 Not Found: @pfelements/pfelement@1.0.0-prerelease.7 which you can safely ignore
  4. ran npm run bootstrap
  5. ran npm install from inside the number web component root.

@@ -0,0 +1,116 @@
/*
Copy link
Contributor

Choose a reason for hiding this comment

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

@chrisdo1 The copyright is no longer needed at the top of the src/pfe-hide-show.js file. You can delete the entire comment here.

Copy link
Contributor

Choose a reason for hiding this comment

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

@castastrophe I have removed the comment.

castastrophe
castastrophe previously approved these changes Jan 21, 2019
Copy link
Contributor

@castastrophe castastrophe left a comment

Choose a reason for hiding this comment

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

Looks good! I added a couple minor issues that we can go back and look at later.


return `
<section>
<h2>Hide-Show</h2>
Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

@starryeyez024 fixed. Thanks!

Copy link
Contributor

@kylebuch8 kylebuch8 left a comment

Choose a reason for hiding this comment

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

Luddite Gary Technological Misbeliever

@kylebuch8 kylebuch8 merged commit 11039f6 into master Jan 23, 2019
@kylebuch8 kylebuch8 deleted the US188751-hide-show-component branch January 23, 2019 21:38
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.

5 participants