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

Module loaders #503

Merged
merged 311 commits into from
Jan 22, 2018
Merged

Module loaders #503

merged 311 commits into from
Jan 22, 2018

Conversation

Benjaki2
Copy link
Collaborator

@Benjaki2 Benjaki2 commented Sep 7, 2017

Resolves #494
Fixes #528
Fixes #673

Depends on nasa-gibs/worldview-components#32

Outstanding Items

Features to be modularized and re-enabled

Update Buster Tests

All tests should be modularized, and the code should be tested against them before that section is marked complete)

  • Core files (fixtures.js, wv.brand-test.js)
  • Data
  • Date
  • Layer
  • Link
  • Map
  • Palettes
  • Proj
  • UI
  • Util

Other Tasks

  • Clean up build scripts
  • Remove converted dependencies (wv.js.json, dependencies, etc.)
  • Move all dependencies from /ext to package.json
  • Make sure all required CSS files are included (some have been removed)
  • Shim or include jscache library in package.json
  • Enable watch mode for JS and CSS build tasks - @localjo Update build scripts #647
  • Make build script work for dev and production environments - @localjo Update build scripts #647

Bugs

  • Fix projection button (Getting the error "Cannot read property 'close' of undefined in projection/projection/ui.js")
  • Fix "Add layer" errors (in layer module)
  • Fix error related to WVC on tab switching
  • The URL state is not updating
  • iCheck boxes are missing iCheck iCheck-checkbox classes on div which is affecting the styling
  • Hovering a layer on the map the includes a colorbar does not update the label in the sidebar. (.wv-running / .wv-active classes not being added from map/runningdata.js)
  • jQuery.easing[this.easing] is not a function error when opening projection switcher.
  • Styling of projection dropdown is off (more spacing and not highlighted)
  • React.createElement: type should not be null, undefined, boolean, or number. It should be a string (for DOM elements) or a ReactClass (for composite components). when trying to open share link button.
  • jQuery UI default stylesheet changing causes a number of elements to appear different.
    -- Text appears darker than before. This can be seen in the Events / Date tab, Measurement setting tab (Add Layers modal),
    -- The "Squash Layer" checkbox in the sidebar has no border and is missing padding/margin right
    -- The radio "check" buttons under Color palettes in the sidebar is missing thick borders
    -- jQuery UI default font-stack is no longer Segoe UI,Arial,sans-serif, it is Verdana,Arial,sans-serif, so the "Add Layers" button looks different
  • Add Layers -> Measurements description drawer button not working (no error displayed)
    Note* This is related to toggle-event being removed in jQuery 1.9+ (https://api.jquery.com/toggle-event/)
  • Map Scale Bug

Cleanup

@ghost ghost assigned Benjaki2 Sep 7, 2017
@ghost ghost added the under development label Sep 7, 2017
@ZachTRice ZachTRice added this to the Release 1.10.0 milestone Sep 8, 2017
@Benjaki2
Copy link
Collaborator Author

Note 2** there are onload functions in wv.util.browser to be reworked into the onload function in wv.main.js

@ghost ghost assigned localjo Sep 19, 2017
@localjo
Copy link
Contributor

localjo commented Sep 19, 2017

Getting this message on npm install;

We're super 😸 excited that you're trying to use ES2015 syntax, but instead of making more yearly presets 😭 , Babel now has a better preset that we recommend you use instead: npm install babel-preset-env --save-dev. preset-env without options will compile ES2015+ down to ES5 just like using all the presets together and thus is more future proof. It also allows you to target specific browsers so that Babel can do less work and you can ship native ES2015+ to user 😎 ! We are also in the process of releasing v7, so please give http://babeljs.io/blog/2017/09/12/planning-for-7.0 a read and help test it out in beta! Thanks so much for using Babel 🙏, please give us a follow on Twitter @babeljs for news on Babel, join slack.babeljs.io for discussion/development and help support the project at opencollective.com/babel

@Benjaki2
Copy link
Collaborator Author

@localjo Yeah I saw that yesterday. Need to switch the preset-package-name

@localjo localjo removed this from the Release 1.10.0 milestone Oct 16, 2017
@ghost ghost assigned ZachTRice Nov 8, 2017
ZachTRice and others added 11 commits November 8, 2017 12:04
Specifically:
* Shimmed d3
* Shimmed Jquery and it's plugins
* Removed `const` that were exported and replaced them with default
* executions
* Added build folder to place store transpiled code
* Removed uglify
* Removed jQuery plugin tags
* Adjusted `export` && `import` naming to make them correspond/work
* A number of bug fixes with help from the linter
Pull in correct ol CSS files with Grunt and update link tags in index.html
npm install automatically updated this
- Target `a` elements with sortable's cancel attribute within the layer `li`
- Create .github default folder for housing github related docs
- Add a link to our waffle.io board to contributing doc
- Minor spellcheck corrections
- Increase validScreenSize width detection to prevent the tour window from appearing on devices with too small of an area for the window to fit correctly.
@Benjaki2 Benjaki2 mentioned this pull request Jan 19, 2018
- Change `left` position direction to `right` as per documentation
- Import `jquery-ui/position` to wvui to prevent unexpected results
@localjo localjo merged commit 0875fea into development Jan 22, 2018
@ghost ghost removed the testing label Jan 22, 2018
@localjo localjo deleted the module-loaders branch January 23, 2018 17:23
This was referenced Jan 23, 2018
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.

4 participants