events.action: opened
events.author: jamesfan
events.text: ## Summary
  
  Addresses #294.
  
  Modifies the Select component to use the Canvas-designed menu instead of the native select menu.
  
  ## Checklist
  
  <!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->
  
  - [ ] branch has been rebased on the latest master commit
  - [ ] tests are changed or added
  - [ ] `yarn test` passes
  - [ ] all (dev)dependencies that the module needs is added to its `package.json`
  - [ ] code has been documented and, if applicable, usage described in README.md
  - [ ] module has been added to `canvas-kit-react` and/or `canvas-kit-css` universal modules, if
        applicable
  - [ ] design approved final implementation
  - [ ] a11y approved final implementation
  - [ ] code adheres to the [API & Pattern guidelines](../API_PATTERN_GUIDELINES.md)
events.title: feat(select): Implement Canvas menu
events.type: issue
events.action: created
events.author: NicholasBoll
events.text: There are several Typescript errors that are preventing a Storybook example from running in Chroma
events.type: comment
events.action: created
events.author: jamesfan
events.text: @NicholasBoll Thanks for the feedback! I'll start addressing these fixes/questions shortly.
events.type: comment
events.action: created
events.author: jamesfan
events.text: @anicholls Awesome, will do!
events.type: comment
events.action: created
events.author: jamesfan
events.text: @NicholasBoll @anicholls
  
  _I'm adding this as a brand new comment rather than adding it to the original thread since it's getting buried in the hidden items. Here's the original thread for reference: https://github.com/Workday/canvas-kit/pull/454#discussion_r395942602_
  
  I've implemented the render prop (`renderOption`) in the latest changes if you'd like to check it out. I've included example code in the README as well as a `Custom Options` story.
  
  It's a slightly modified version of Nicholas's proposed API -- I opted to have `renderOption` only handle the content of the `<SelectOption>` and not the `<SelectOption>` itself for simplicity.
  
  It would be nice if we could use `renderOption` to render the actual Select itself (which is actually a button), but since it's a `<button>` we're limited in what we can include in its content. Something for a later PR perhaps.
events.type: comment
events.action: created
events.author: NicholasBoll
events.text: It's unfortunate how these stories show up in documentation: 
  <img width="423" alt="Screen Shot 2020-04-23 at 5 11 49 PM" src="https://user-images.githubusercontent.com/338257/80158382-937feb00-8585-11ea-945f-00b42956c8fa.png">
events.type: comment
events.action: created
events.author: jamesfan
events.text: Agreed, do you think it would be preferable to remove `defaultWithSimpleOptionsComponent` and replace it with the actual component code (at the expense of adding duplicate code and making it a bit more tedious to update stories)?
events.type: comment
events.action: created
events.author: NicholasBoll
events.text: Yes. A story's implementation is meant to show an example use-case that is as copy/paste-friendly as possible. We want the source to be easy to understand.
  
  Ideally someone could just copy/paste everything in the "source" tab of the component and make their modifications, but unfortunately Storybook's source plugin doesn't do that very well. Despite the plugin's shortcomings, I'd like to make story source as good as we can without modifying Storybook or the plugins directly. Perhaps we'll fix the source at some point in the future.
  
  Related: #597
events.type: comment
data_with_bots.bot: <p align="center">
    <br />
    <img src="https://assets.cypress.io/github-pr-comment-icons/cypress-logo.png" height="32" />
  </p>
  
  ---
  
  ### Test summary
  <a href="https://dashboard.cypress.io/projects/odida5/runs/1008/specs"><img src="https://assets.cypress.io/github-pr-comment-icons/green-check-mark.png" width="14" height="14" title="Passed" /></a> <b title="Passed">155</b>  &bull; <a href="https://dashboard.cypress.io/projects/odida5/runs/1008/failures"><img src="https://assets.cypress.io/github-pr-comment-icons/red-x.png" width="14" height="14" title="Failed" /></a> <b title="Failed">0</b>  &bull; <img src="https://assets.cypress.io/github-pr-comment-icons/gray-circle.png" width="14" height="14" title="Tests that did not run due to a developer annotating a test with .skip" /> <b title="Pending">0</b> &bull; <img src="https://assets.cypress.io/github-pr-comment-icons/yellow-triangle.png" width="14" height="14" title="Tests that did not run due to a failure in a mocha hook" /> <b title="Skipped">0</b>
  
  ---
  
  ### Run details
  | | |
  | --- | --- |
  | **Project** | canvas-kit |
  | **Status** | Passed |
  | **Commit** | ea338fbca9 <span style="cursor:help;" title="Your CI provider generated this commit by merging this branch into the target branch.">ℹ️</span> |
  | **Started** | Feb 25, 2020 6:24 AM |
  | **Ended** | Feb 25, 2020 6:27 AM |
  | **Duration** | 03:35 <a href="https://on.cypress.io/parallelization" title="Enable parallelization to reduce run duration">💡</a> |
  | **OS** | Linux Ubuntu Linux - 16.04 |
  | **Browser** | Electron 78 <img src="https://cdnjs.cloudflare.com/ajax/libs/browser-logos/48.0.4/electron/electron_32x32.png" width="16" height="16" /> |
  
  [View run in Cypress Dashboard ➡️](https://dashboard.cypress.io/projects/odida5/runs/1008/specs)
  
  
  
  ---
  
  This comment has been generated by **cypress-bot** as a result of this project's GitHub integration settings. You can manage this integration in [this project's settings in the Cypress Dashboard](https://dashboard.cypress.io/projects/odida5/settings)
data_with_bots.meta: This message directly preceded comment from bot *cypress[bot]* by author *jamesfan*:
data_with_bots.text: @NicholasBoll Thanks for the feedback! I'll start addressing these fixes/questions shortly.
data_without_bots.meta: This message directly preceded comment from bot *cypress[bot]* by author *jamesfan*:
data_without_bots.text: @NicholasBoll Thanks for the feedback! I'll start addressing these fixes/questions shortly.