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

Toggle Grid Display #147

Merged
merged 4 commits into from
Jun 21, 2017
Merged

Conversation

theduckylittle
Copy link
Member

Adds the ability to min/maximize the grid. The button even
does a fancy icon change.

Other included changes:

  1. This also cleans up some of the differing CSS between the toolbar
    and the grid. Longer term, we should initiate a project that
    reuses more components but for now, it seemed like a good idea
    to at least fix the CSS better.
  2. Fix for the missing highlight functions (merge or stupidity caused them to get chucked, I'd imagine).
  3. Fix for module.exports being undefined. The polyfill now is smarter and does a typeof check on module to ensure that the module 'api' exists before trying to use that object.

refs: #124

Adds the ability to min/maximize the grid. The button even
does a fancy icon change.

This also cleans up some of the differing CSS between the toolbar
and the grid.  Longer term, we should initiate a project that
reuses more components but for now, it seemed like a good idea
to at least fix the CSS better.
@klassenjs klassenjs merged commit ac8fd44 into geomoose:master Jun 21, 2017
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.

2 participants