Permalink
Browse files

Year 2017 and fix lint to OL2

  • Loading branch information...
1 parent a333591 commit 35ff0e251a890c52728a928499808185670c8b52 @jorix committed Jan 10, 2017
View
@@ -1,4 +1,4 @@
-Copyright 2011-2016 by Xavier Mamano, http://github.com/jorix/OL-FeaturePopups
+Copyright 2011-2017 by Xavier Mamano, http://github.com/jorix/OL-FeaturePopups
Permission is hereby granted, free of charge, to any person obtaining
a copy of this software and associated documentation files (the
@@ -22,14 +22,14 @@ <h1 id="title">Popups issues: autosize causes extra padding</h1>
<div id="shortdesc">
<dl>
<dt>Extra padding on bottom & right (fixed with
- <a href="https://github.com/openlayers/openlayers/pull/533">#544</a>):
+ <a href="https://github.com/openlayers/ol2/pull/533">#544</a>):
</dt>
<dd>Popups that uses auto size displays a extra padding on
bottom & right, using browsers that uses W3C box model:
Chrome, FF, Safari, Opera and
IE9 not compatibility mode.</dd>
<dt>Not correctly calculated the position of close box (fixed with
- <a href="https://github.com/openlayers/openlayers/pull/533">#533</a>):
+ <a href="https://github.com/openlayers/ol2/pull/533">#533</a>):
</dt>
<dd>Using IE except IE9 not compatibility mode.</dd>
<dt>A border causes addicional padding (not yet proposed a patch):</dt>
@@ -25,14 +25,14 @@ <h1 id="title">??</h1>
<div id="shortdesc">
<dl>
<dt>Extra padding on bottom & right (fixed with
- <a href="https://github.com/openlayers/openlayers/pull/533">#544</a>):
+ <a href="https://github.com/openlayers/ol2/pull/533">#544</a>):
</dt>
<dd>Popups that uses auto size displays a extra padding on
bottom & right, using browsers that uses W3C box model:
Chrome, FF, Safari, Opera and
IE9 not compatibility mode.</dd>
<dt>Not correctly calculated the position of close box (fixed with
- <a href="https://github.com/openlayers/openlayers/pull/533">#533</a>):
+ <a href="https://github.com/openlayers/ol2/pull/533">#533</a>):
</dt>
<dd>Using IE except IE9 not compatibility mode.</dd>
<dt>A border causes addicional padding (not yet proposed a patch):</dt>
@@ -26,14 +26,14 @@ <h1 id="title">??</h1>
<div id="shortdesc">
<dl>
<dt>Extra padding on bottom & right (fixed with
- <a href="https://github.com/openlayers/openlayers/pull/533">#544</a>):
+ <a href="https://github.com/openlayers/ol2/pull/533">#544</a>):
</dt>
<dd>Popups that uses auto size displays a extra padding on
bottom & right, using browsers that uses W3C box model:
Chrome, FF, Safari, Opera and
IE9 not compatibility mode.</dd>
<dt>Not correctly calculated the position of close box (fixed with
- <a href="https://github.com/openlayers/openlayers/pull/533">#533</a>):
+ <a href="https://github.com/openlayers/ol2/pull/533">#533</a>):
</dt>
<dd>Using IE except IE9 not compatibility mode.</dd>
<dt>A border causes addicional padding (not yet proposed a patch):</dt>
View
@@ -87,7 +87,7 @@ The problems were fixed:
* Remove extra padding: [autoSize-padding-dev.html][autoSize-padding-dev]
* Narrow content when displaying scroll bar: [popups-overflow-dev.html](http://jorix.github.com/OL-FeaturePopups/OL-Popup-issues/popups-overflow-dev.html)
* Adjust the insertion point using FramedCloud popup: [popups-FramedCloud-insertion-point.html](http://jorix.github.com/OL-FeaturePopups/OL-Popup-issues/popups-FramedCloud-insertion-point.html)
- * Avoid breaking the "OpenLayers.String.format" execution [#OL-631](https://github.com/openlayers/openlayers/pull/631)
+ * Avoid breaking the "OpenLayers.String.format" execution [#OL-631](https://github.com/openlayers/ol2/pull/631)
Patches to fix other problems not yet proposed to OL, but applied in this code:
* Chrome+ABP does not consider the size of images: [popups-img-chrome-dev.html](http://jorix.github.com/OL-FeaturePopups/OL-Popup-issues/popups-img-chrome-dev.html)
@@ -1,4 +1,4 @@
-/* Copyright 2011-2016 by Xavier Mamano http://github.com/jorix/OL-FeaturePopups
+/* Copyright 2011-2017 by Xavier Mamano http://github.com/jorix/OL-FeaturePopups
* Published under MIT license. */
/**
@@ -1,4 +1,4 @@
-/* Copyright 2011-2016 by Xavier Mamano http://github.com/jorix/OL-FeaturePopups
+/* Copyright 2011-2017 by Xavier Mamano http://github.com/jorix/OL-FeaturePopups
* Published under MIT license. */
/**
@@ -1,4 +1,4 @@
-/* Copyright (c) 2006-2016 by OpenLayers Contributors (see authors.txt for
+/* Copyright (c) 2006-20xx by OpenLayers Contributors (see authors.txt for
* full list of contributors). Published under the 2-clause BSD license.
* See license.txt in the OpenLayers distribution or repository for the
* full text of the license. */
@@ -15,24 +15,24 @@ autosize. For more details see "../README.md"
Patches included:
pending to review by OL team:
- https://github.com/openlayers/openlayers/pull/507 Get right size when
+ https://github.com/openlayers/ol2/pull/507 Get right size when
force height or width.
- https://github.com/openlayers/openlayers/pull/485 Get right scrollbar
+ https://github.com/openlayers/ol2/pull/485 Get right scrollbar
width.
- https://github.com/openlayers/openlayers/pull/533 Using IE no parent
+ https://github.com/openlayers/ol2/pull/533 Using IE no parent
is a object!
- https://github.com/openlayers/openlayers/pull/544 Extra padding.
- https://github.com/openlayers/openlayers/pull/565 Fractional size
+ https://github.com/openlayers/ol2/pull/544 Extra padding.
+ https://github.com/openlayers/ol2/pull/565 Fractional size
using IE9.
- https://github.com/openlayers/openlayers/pull/566 Take into account
+ https://github.com/openlayers/ol2/pull/566 Take into account
scroll bar to calculate min/max size.
- https://github.com/openlayers/openlayers/pull/613 FramedCloud Popup:
+ https://github.com/openlayers/ol2/pull/613 FramedCloud Popup:
adjust the insertion point.
fixed in 2.12:
- https://github.com/openlayers/openlayers/pull/505
+ https://github.com/openlayers/ol2/pull/505
other patches included but are pending publication as "Pull Rquest" on OL:
* Chrome+ABP does not consider the size of images
- (https://github.com/openlayers/openlayers/issues/595).
+ (https://github.com/openlayers/ol2/issues/595).
* Consider the border on `contentDisplayClass`.
* Margin should be considered when padding on `contentDisplayClass`
is zero.

0 comments on commit 35ff0e2

Please sign in to comment.