leaflet-blurred-location icon indicating copy to clipboard operation
leaflet-blurred-location copied to clipboard

getblurryscale() undefined error resolved AND changing LBL syntax

Open sagarpreet-chadha opened this issue 5 years ago • 9 comments

Fixes #194 , #193 😄 .

Work in Progress 🚧 👷 !

sagarpreet-chadha avatar Apr 08 '19 06:04 sagarpreet-chadha

Hi @jywarren , This PR changes the syntax of LBL declaration to the same way a standard Leaflet object is added .

var map123 = L.map('map').setView([23,77] , 3) ;

var blurredLocation = new BlurredLocation(options) ; blurredLocation.addTo(map123) ;

Kindly review 😄 !

sagarpreet-chadha avatar Apr 08 '19 16:04 sagarpreet-chadha

Awesome! Let's update the docs too!

jywarren avatar Apr 09 '19 00:04 jywarren

Is this a breaking change regarding https://semver.org?

jywarren avatar Apr 09 '19 00:04 jywarren

If not, let's merge! If so, maybe we need to update the version number first. Thanks!!!

jywarren avatar Apr 09 '19 00:04 jywarren

If you upload to Github you should be able to copy the URL to your proposal!

On Mon, Apr 8, 2019 at 8:24 PM Ashutosh Kumar [email protected] wrote:

@jywarren https://github.com/jywarren I am uploading this gif to my proposal but was unable to do so. any idea?? [image: crdtpng] https://user-images.githubusercontent.com/25272573/55765107-ca26b880-5a8b-11e9-8429-a8dc7474c075.gif

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/publiclab/leaflet-blurred-location/pull/195#issuecomment-481055078, or mute the thread https://github.com/notifications/unsubscribe-auth/AABfJ6fqB6UdenuXXW3hRSuNdPjf5I5lks5ve92sgaJpZM4chaTI .

jywarren avatar Apr 09 '19 00:04 jywarren

@jywarren is this gif fine for demo implementation of synchronous editing.

igniteeng000 avatar Apr 09 '19 00:04 igniteeng000

Hi @jywarren , updated readme and version number is already in the PR 😄 .

I have changed the version number from 1.2.8 to 1.3.0 but i guess we are introducing backwards incompatible change , so we should change version to 2.0.0 according to https://semver.org/ .

Thank you !

PS : I would be writing detailed documentation in accordance to https://github.com/publiclab/leaflet-blurred-location-display/issues/64 later .

sagarpreet-chadha avatar Apr 09 '19 07:04 sagarpreet-chadha

+1 v2.0!!! Thank you!!!

jywarren avatar Apr 18 '19 19:04 jywarren

Done 👍 . Should we merge this ? Thanks You !

sagarpreet-chadha avatar Apr 19 '19 07:04 sagarpreet-chadha