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

Merge 2.4 into master #4914

Merged
merged 36 commits into from
May 29, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
4647642
Port Offline feature from older ngeo version
gberaudo Nov 1, 2018
a140465
Make eslint happy
gberaudo Nov 1, 2018
c4e92a7
Simplify imports
gberaudo Nov 27, 2018
8531beb
Use same signature for webpack.dev.js and webpack.prod.js
gberaudo Dec 12, 2018
1469f6e
Revert "Use same signature for webpack.dev.js and webpack.prod.js"
gberaudo Dec 13, 2018
30dcb78
Allow passing a plain service to the Offline service manager
gberaudo Dec 21, 2018
2580af7
Merge branch 'allow_overriding_vector_encoder' into port_offline_feature
gberaudo Apr 3, 2019
1e2571a
Fix HTML indentation
fredj May 10, 2019
be7a208
Set loading-mask to the whole screen
fredj May 10, 2019
778759a
Merge pull request #4891 from camptocamp/GSGMF-851
fredj May 13, 2019
81c1b43
Small documentation improvements
sbrunner May 13, 2019
b8f7d88
Add tag to publish
sbrunner May 15, 2019
b0a5e70
Merge pull request #4898 from camptocamp/fix-tag
sbrunner May 16, 2019
fcb9582
Merge pull request #4897 from camptocamp/doc
sbrunner May 20, 2019
d8a96ee
Remove wrong height css value for ms-thumb
fredj May 20, 2019
c19561f
Add improvements for apps
gberaudo Feb 1, 2019
4c27b6c
Merge pull request #4348 from camptocamp/port_offline_feature
llienher May 21, 2019
86fefb0
Merge pull request #4902 from camptocamp/GSGMF-966
fredj May 21, 2019
ba4ce97
Merge branch '2.3' into merge_in_2.3
gberaudo May 22, 2019
0596caf
Fix various typescript errors
llienher May 22, 2019
51a1bb9
Make color property optional
fredj May 23, 2019
cc20897
Merge pull request #4908 from camptocamp/GSGMF-967
fredj May 23, 2019
e344956
Fix more typing issues
gberaudo May 24, 2019
8afd470
Change the cursor to 'move' on modal header hovering
fredj May 27, 2019
cd757c4
Merge pull request #4910 from camptocamp/GSGMF-979
fredj May 27, 2019
7ec5eda
Add remaning typescript fixes
gberaudo May 27, 2019
a9def4c
Fix eslint
gberaudo May 27, 2019
21fef29
Aditionnal fixes
gberaudo May 27, 2019
bbbdd32
api: reverse to order of the coordinates to be compatible with the ol…
fredj May 8, 2019
330fc72
Merge pull request #4912 from camptocamp/GSGMF-942
fredj May 27, 2019
f1bfde0
Fix localforage imports
gberaudo May 27, 2019
f1c4c6c
Some more type fixes
gberaudo May 27, 2019
c949bda
Last fixes (hopefully)
gberaudo May 27, 2019
794a0c2
Merge pull request #4913 from camptocamp/merge_in_2.3
gberaudo May 28, 2019
9317e5b
Merge remote-tracking branch 'origin/2.4'
gberaudo May 28, 2019
d75b998
Use @return
gberaudo May 29, 2019
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
5 changes: 1 addition & 4 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,10 +65,7 @@ script:
# Generate API doc
- npm install --prefix openlayers_src https://api.github.com/repos/openlayers/openlayers/tarball/v5.3.1
- '! (npm run typecheck|sed -r "s/\x1B\[([0-9]{1,2}(;[0-9]{1,2})?)?[mGK]//g"|grep ^src)'
- '! (npm run typecheck|grep api)'
- '! (npm run typecheck|grep gmf)'
- '! (npm run typecheck|grep test/)'
- '! (npm run typecheck|grep examples)'
- '! (npm run typecheck|grep -E "api|gmf|test/|examples")'
- npm run doc
- npm run build-api

Expand Down
4 changes: 2 additions & 2 deletions api/src/Map.js
Original file line number Diff line number Diff line change
Expand Up @@ -377,7 +377,7 @@ class Map {
/**
* @param {string[]} keys Keys.
* @param {Array<*>} values Values.
* @returns {Object<string, *>} Object.
* @return {Object<string, *>} Object.
* @private
* @hidden
*/
Expand All @@ -394,7 +394,7 @@ function zip(keys, values) {
/**
* @param {Object<string, *>} obj Object.
* @param {string[]} keys keys.
* @returns {Object<string, *>} Object.
* @return {Object<string, *>} Object.
* @private
* @hidden
*/
Expand Down
8 changes: 4 additions & 4 deletions api/src/Themes.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ let themesPromise;

/**
* @hidden
* @returns {Promise<import('gmf/themes.js').GmfThemesResponse>} Promise
* @return {Promise<import('gmf/themes.js').GmfThemesResponse>} Promise
*/
function getThemesPromise() {
if (!themesPromise) {
Expand All @@ -35,7 +35,7 @@ let overlayDefPromise;

/**
* @hidden
* @returns {Promise<Array<TileLayer|ImageLayer|GroupLayer>>} Promise
* @return {Promise<Array<TileLayer|ImageLayer|GroupLayer>>} Promise
*/
export function getBackgroundLayers() {
return getThemesPromise().then((themes) => {
Expand Down Expand Up @@ -107,7 +107,7 @@ const overlayDefs = new Map();

/**
* @hidden
* @returns {Promise<Map<string, overlayDefinition>>} Promise
* @return {Promise<Map<string, overlayDefinition>>} Promise
*/
export function getOverlayDefs() {
if (!overlayDefPromise) {
Expand All @@ -128,7 +128,7 @@ export function getOverlayDefs() {
* @param {Object} config Config
* @param {import('gmf/themes.js').GmfOgcServers} ogcServers OGC servers
* @param {import('gmf/themes.js').GmfOgcServer} [opt_ogcServer] OGC server
* @returns {void}
* @return {void}
* @hidden
*/
export function writeOverlayDefs(config, ogcServers, opt_ogcServer) {
Expand Down
4 changes: 2 additions & 2 deletions buildtools/travis-npm-publish.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ then
fi
if [ "$PACKAGE_VERSION" = "$TRAVIS_TAG" ]
then
$RUN npm publish $TAG
$RUN npm publish --tag=version-${MAIN_BRANCH} $TAG
else
echo "Skipping publication, the travis tag and package version differ"
fi
Expand All @@ -27,7 +27,7 @@ then
then
echo "Publish daily version"
$RUN npm install --no-save fluid-publish
$RUN node_modules/.bin/fluid-publish devTag="latest"
$RUN node_modules/.bin/fluid-publish devTag="version-${MAIN_BRANCH}-latest"
fi
fi
fi
2 changes: 1 addition & 1 deletion contribs/gmf/apps/desktop/index.html.ejs
Original file line number Diff line number Diff line change
Expand Up @@ -289,7 +289,7 @@
<ngeo-modal
ngeo-modal-closable="false"
ng-model="mainCtrl.userMustChangeItsPassword">
<div class="modal-header">
<div class="modal-header ui-draggable-handle">
<h4 class="modal-title">
{{'You must change your password' | translate}}
</h4>
Expand Down
2 changes: 1 addition & 1 deletion contribs/gmf/apps/desktop_alt/index.html.ejs
Original file line number Diff line number Diff line change
Expand Up @@ -333,7 +333,7 @@
<gmf-share ng-if="mainCtrl.modalShareShown" gmf-share-email="false"></gmf-share>
</ngeo-modal>
<ngeo-modal ng-model="disclaimerVisibility">
<div class="modal-header">
<div class="modal-header ui-draggable-handle">
<button type="button" class="close" data-dismiss="modal" aria-hidden="true">&times;</button>
</div>
<div class="modal-body">
Expand Down
8 changes: 4 additions & 4 deletions contribs/gmf/apps/iframe_api/index.html.ejs
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@
<% } %>
</head>
<body>
<div ng-show="mainCtrl.loading" class="loading-mask">
<i class="fa fa-spinner fa-spin spinner-loading-mask"></i>
</div>
<div ng-show="mainCtrl.loading" class="loading-mask">
<i class="fa fa-spinner fa-spin spinner-loading-mask"></i>
</div>
<main>
<div class="gmf-app-map-container" ng-class="{'gmf-app-infobar-active': mainCtrl.showInfobar}">
<ngeo-displaywindow
Expand Down Expand Up @@ -76,7 +76,7 @@
<ngeo-modal
ngeo-modal-closable="false"
ng-model="mainCtrl.userMustChangeItsPassword">
<div class="modal-header">
<div class="modal-header ui-draggable-handle">
<h4 class="modal-title">
{{'You must change your password' | translate}}
</h4>
Expand Down
10 changes: 4 additions & 6 deletions contribs/gmf/apps/mobile/index.html.ejs
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,9 @@
<body ng-class="{'gmf-mobile-nav-is-visible': mainCtrl.navIsVisible(),
'gmf-mobile-nav-left-is-visible': mainCtrl.leftNavVisible,
'gmf-mobile-nav-right-is-visible': mainCtrl.rightNavVisible}">
<div ng-show="mainCtrl.loading" class="loading-mask">
<i class="fa fa-spinner fa-spin spinner-loading-mask"></i>
&nbsp;
<img src="<%=require("./image/logo.png")%>" />
</div>
<div ng-show="mainCtrl.loading" class="loading-mask">
<i class="fa fa-spinner fa-spin spinner-loading-mask"></i>
</div>
<main ng-class="{'gmf-search-is-active': mainCtrl.searchOverlayVisible}">
<gmf-map
gmf-map-map="mainCtrl.map"
Expand Down Expand Up @@ -190,7 +188,7 @@
</nav>
<ngeo-modal
ng-model="mainCtrl.userMustChangeItsPassword">
<div class="modal-header">
<div class="modal-header ui-draggable-handle">
<h4 class="modal-title">
{{'You must change your password' | translate}}
</h4>
Expand Down
8 changes: 3 additions & 5 deletions contribs/gmf/apps/mobile_alt/index.html.ejs
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,9 @@
<body ng-class="{'gmf-mobile-nav-is-visible': mainCtrl.navIsVisible(),
'gmf-mobile-nav-left-is-visible': mainCtrl.leftNavVisible,
'gmf-mobile-nav-right-is-visible': mainCtrl.rightNavVisible}">
<div ng-show="mainCtrl.loading" class="loading-mask">
<i class="fa fa-spinner fa-spin spinner-loading-mask"></i>
&nbsp;
<img src="<%=require("./image/logo.png")%>" />
</div>
<div ng-show="mainCtrl.loading" class="loading-mask">
<i class="fa fa-spinner fa-spin spinner-loading-mask"></i>
</div>
<main ng-class="{'gmf-search-is-active': mainCtrl.searchOverlayVisible}">
<gmf-map
gmf-map-map="mainCtrl.map"
Expand Down
2 changes: 1 addition & 1 deletion contribs/gmf/examples/common_dependencies.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import 'gmf/sass/vars.scss'
import 'gmf/sass/vars.scss';
import 'jquery';
import 'angular';
import 'angular-gettext';
Expand Down
2 changes: 1 addition & 1 deletion contribs/gmf/src/authentication/component.html
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@

<ngeo-modal
ng-model="$ctrl.resetPasswordModalShown">
<div class="modal-header">
<div class="modal-header ui-draggable-handle">
<button type="button"
class="close"
data-dismiss="modal"
Expand Down
2 changes: 1 addition & 1 deletion contribs/gmf/src/authentication/component.js
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ function gmfAuthenticationTemplateUrl($element, $attrs, gmfAuthenticationTemplat
* <ngeo-modal
* ngeo-modal-closable="false"
* ng-model="mainCtrl.userMustChangeItsPassword">
* <div class="modal-header">
* <div class="modal-header ui-draggable-handle">
* <h4 class="modal-title">
* {{'You must change your password' | translate}}
* </h4>
Expand Down
8 changes: 4 additions & 4 deletions contribs/gmf/src/backgroundlayerselector/component.js
Original file line number Diff line number Diff line change
Expand Up @@ -66,11 +66,11 @@ function gmfBackgroundlayerselectorTemplateUrl($element, $attrs, gmfBackgroundla
*
* Used metadata:
*
* * thumbnail: The URL used for the icon.
* * `thumbnail`: The URL used for the icon.
*
* Used functionnalities:
* Used functionalities:
*
* * default_basemap: Base maps to use by default.
* * `default_basemap`: Base maps to use by default.
*
* @htmlAttribute {import("ol/Map.js").default=} gmf-backgroundlayerselector-map The map.
* @htmlAttribute {string} gmf-backgroundlayer-opacity-options The opacity slider options.
Expand Down Expand Up @@ -207,7 +207,7 @@ Controller.prototype.handleThemesChange_ = function() {
/**
* Getter/setter for background layer overlay, used by opacity slider.
* @param {?number} val The opacity.
* @returns {number} The background layer opacity.
* @return {number} The background layer opacity.
*/
Controller.prototype.getSetBgLayerOpacity = function(val) {
if (!this.opacityLayer) {
Expand Down
7 changes: 2 additions & 5 deletions contribs/gmf/src/controllers/AbstractAppController.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,12 +53,9 @@ import {ThemeEventType} from 'gmf/theme/Manager.js';
/**
* Application abstract controller.
*
* This file includes `goog.require` for base components/directives used
* by the HTML page and the controller to provide the configuration.
* Used functionalities:
*
* Used functionnalities:
*
* * open_panel: When set, contains the name of the panel to open upon loading an application.
* * `open_panel`: When set, contains the name of the panel to open upon loading an application.
* Note: although this is a list, only one can be defined.
*
*
Expand Down
1 change: 1 addition & 0 deletions contribs/gmf/src/controllers/AbstractDesktopController.js
Original file line number Diff line number Diff line change
Expand Up @@ -228,6 +228,7 @@ export class AbstractDesktopController extends AbstractAPIController {
* If, after resizing, the size of the data panel would be too big,
* resize it as well.
* @private
* @hidden
*/
setDataPanelMaxResizableWidth_() {

Expand Down