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

Move some services to misc module (second pack) #3303

Merged
merged 4 commits into from
Dec 21, 2017

Conversation

ger-benjamin
Copy link
Member

@ger-benjamin ger-benjamin commented Dec 20, 2017

For GSGMF-214

Then, we still need to move that in the extra "misc" module (in another PR):

  • src/directives/btngroup.js
  • src/directives/control.js
  • src/directives/filereader.js
  • src/directives/sortable.js
  • contribs/gmf/src/directives/swipe.js
  • src/directives/colorpicker.js
  • src/directives/datepicker.js

@ger-benjamin ger-benjamin changed the title [WIP] Move services to misc module [WIP] Move some services to misc module (second pack) Dec 20, 2017
@ger-benjamin ger-benjamin self-assigned this Dec 20, 2017
@ger-benjamin ger-benjamin changed the title [WIP] Move some services to misc module (second pack) Move some services to misc module (second pack) Dec 21, 2017
@@ -1,7 +1,8 @@
goog.provide('ngeo.DecorateInteraction');
goog.provide('ngeo.misc.decorate');
Copy link
Member

Choose a reason for hiding this comment

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

nice! we should move the layer decorator here (#3309)

Copy link
Member Author

@ger-benjamin ger-benjamin Dec 21, 2017

Choose a reason for hiding this comment

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

Yes !
I add a card about that: https://jira.camptocamp.com/browse/GSGMF-217

@@ -15,17 +16,9 @@ goog.require('ngeo');
*
* See our live example: [../examples/interactiontoggle.html](../examples/interactiontoggle.html)
Copy link
Member

Choose a reason for hiding this comment

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

remove this line (the example is gone)

Copy link
Member Author

@ger-benjamin ger-benjamin Dec 21, 2017

Choose a reason for hiding this comment

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

I'll remove it in the third (and last) "misc" pack

@ger-benjamin ger-benjamin merged commit bf5edc5 into master Dec 21, 2017
@ger-benjamin ger-benjamin deleted the move_services_to_misc_module branch December 21, 2017 13:34
@sbrunner sbrunner added this to the 2.3 milestone Apr 12, 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.

3 participants