Reference: https://github.com/blackbaud/skyux2/blob/master/src/modules/animation/slide.ts

When hooking up ngModel to the checkbox with a default value = true the checkbox component has a checked property value = true but does not display as checked.

For an example please look at checkbox 2 on the demo.

Additionally, one should be able to initialize the component in a checked and disabled state. So please consider that use case when updating. Thanks!

Navbar styles in firefox seem to be slightly off when there is a navbar item followed by a navbar dropdown

Browser original width:

Browser made smaller: note one tile has disappeared

Browser back to original size: both tiles end up in right column

We have provided both single and multicolumn layouts in the config:

layout: {
  singleColumn: {
    tiles: [
      { id: 'moon-grant-summary-tile', isCollapsed: false },
      { id: 'moon-grant-summary-tile', isCollapsed: false }
    ]
  },
  multiColumn: [
    {
      tiles: [
        { id: 'moon-grant-summary-tile', isCollapsed: false }
      ]
    },
    {
      tiles: [
        { id: 'moon-grant-summary-tile', isCollapsed: false }
      ]
    }
  ]
}

When viewing the HTML report generated by Istanbul during unit test runs, the code coverage displayed is for the transpiled JS code rather than the original TS code. In CodeCov the TS code is displayed but the highlighted lines match up with the transpiled JS code. We need to figure out a way to use something like remap-instanbul to map coverage back to the original TS file.

There's a thread on the angular2-webpack-starter repo on which the SKY UX 2 project is based that describes difficulties integrating remap-istanbul with their project.

with npm module version 2.0.0-alpha.7:

There is a ref to a missing file.

ERROR in .//blackbaud-skyux2/dist/modules/page-summary/page-summary-alert/page-summary-alert.component.js
Module not found: Error: Can't resolve './page-summary-alert.component.scss' in '/Users/joshgerdes/Development/microedge-moonshot-ui/node_modules/blackbaud-skyux2/dist/modules/page-summary/page-summary-alert'
@ .//blackbaud-skyux2/dist/modules/page-summary/page-summary-alert/page-summary-alert.component.js 10:21-67
@ .//blackbaud-skyux2/dist/modules/page-summary/page-summary-alert/index.js
@ .//blackbaud-skyux2/dist/modules/page-summary/index.js
@ ./~/blackbaud-skyux2/dist/core.js
@ ./src/app/app.module.ts
@ ./src/app/index.ts
@ ./src/main.ts
@ multi main

This component is usually associated with an actionable element (button, link, etc.) and it would be helpful to be able to disable this component's click functionality via an attribute.

If you run npm run watch:test, the tests run once and then the process is terminated. The process should continue to run so that subsequent file changes can trigger test runs as well as so debugging the tests is possible by launching http://localhost:9876/debug.html in a web browser.

The SKY UX 1 file drop control allowed users to drag and drop folders to upload. We need to determine if this is necessary and if so, we need to build in the recursive folder crawling functionality necessary to do that. For reference, the old library we used accomplished this using this piece of code.

None of the stylings on the Sky Checkbox work when using the sky checkbox from the node package. They do work when pulling down this repo and running the demo.

This includes both the single and multiple variants.

This has a dependency on porting the checklist component

Right now we are using file reader dataURLs to get a url for the preview of an upload item. Instead we might want to look into using the window..URL.createObjectURL function to create an object URL instead of the lengthy data urls

The old tiles kept their width at 50% in this case. I started looking at this. It looks like add width: 50%; to the class sky-tile-dashboard-layout-multi makes this behave better. I also noticed that the old tiles used the bootstrap grid to get that, but it seems like you're trying to avoid getting overly dependent on bootstrap for this version.

Thanks for the help.

We need to figure out the right strategy for implementing visual regression tests. While what we have in SKY UX 1 works, the tests are relatively slow. Can we speed these up by perhaps putting every element on one page and showing/hiding them during the test rather than navigating to a new page for every test?

Unable to properly add a reference to this repository via npm install currently. Seems to only grab CHANGELOG/LICENSE/README and package.json, then blows up.

Using a git submodule in the interim to access your scss/components from our prototype app.

The page summary and tile dashboard end up sharing the same instance of SkyMediaQueryService and so the listener in tile dashboard doesn't get registered.

This is an accessibility rule required when an element has the check role

If you remove the problematic call to remap-istanbul cited in #52 so that the watch test works, the resulting code in the browser is the transpiled JavaScript, not the original TypeScript as it was before #49 was merged. You can see this by running npm run watch:test, then opening http://localhost:9876/debug.html in a web browser and using the browser's developer tools to view the source files.

After cloning the repository and then running npm install and npm start and navigating to http://localhost:3000 the list of examples does load, but after clicking on any of them, an error like this is shown in the browser console and the loading text remains on the screen indefinitely

client?843a:48 multi ./demo/src/tile
Module not found: Error: Cannot resolve 'file' or 'directory' ./demo/src/./demo/src/tile/app.component.ts in C:\Users\brad.woods\Documents\sky-pages\skyux2
resolve file
  C:\Users\brad.woods\Documents\sky-pages\skyux2\demo\src\demo\src\tile\app.component.ts doesn't exist
  C:\Users\brad.woods\Documents\sky-pages\skyux2\demo\src\demo\src\tile\app.component.ts.ts doesn't exist
  C:\Users\brad.woods\Documents\sky-pages\skyux2\demo\src\demo\src\tile\app.component.ts.js doesn't exist
resolve directory
  C:\Users\brad.woods\Documents\sky-pages\skyux2\demo\src\demo\src\tile\app.component.ts doesn't exist (directory default file)
  C:\Users\brad.woods\Documents\sky-pages\skyux2\demo\src\demo\src\tile\app.component.ts\package.json doesn't exist (directory description file)
[C:\Users\brad.woods\Documents\sky-pages\skyux2\demo\src\demo\src\tile\app.component.ts]
[C:\Users\brad.woods\Documents\sky-pages\skyux2\demo\src\demo\src\tile\app.component.ts.ts]
[C:\Users\brad.woods\Documents\sky-pages\skyux2\demo\src\demo\src\tile\app.component.ts.js]
 @ multi ./demo/src/tile

The tile folder has too many files. Break these up into subfolders.

When using the Sky checkbox component, if you check and disable the checkbox, it looks no different than when it is checked and active; You still see just the check mark with a blue background color, as if the checkbox was still active. Proper behavior should be that you see the check mark with a gray background color, the same one when it's disabled and unchecked.

If we are attempting to use SKY Pages for SKY UX 2 documentation, we need to ensure that it is able to be properly searched/crawled by google. If this is not the case, we need to come up with alternatives.

There's a bug when the content behind the modal is scrollable which allows users to scroll beyond the modal mask