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

overlay point support and other improvements and fixes #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

overlay point support and other improvements and fixes #1

wants to merge 3 commits into from

Conversation

AndrewArace
Copy link

Added floorplan.maxZoom() getter/setter, for modifying the depth you
could zoom in (i was using large detailed SVG backdrops for the imageLayer)

Fixed project name in build.xml (I don't know what "position-analytics" was, but I assume it was a copy-paste typo from another project)

Recompiled min.js

(sorry for the tabs/spaces diffs)

Added floorplan.maxZoom() getter/setter, for modifying the depth you
could zoom in.
Fixed project name in build.xml
Recompiled min.js
@AndrewArace
Copy link
Author

If you accept it, I'll handled adding to the API wiki page.

floorplan - added layersWidth() assignment to make map tools window
bigger
overlay - added support for standalone points
overlay - fixed bug in select callback
overlay - added dragging logic for click versus drag
rebuilt min.js
@AndrewArace
Copy link
Author

floorplan - added layersWidth() assignment to make map tools window
bigger
overlay - added support for standalone points
overlay - fixed bug in select callback
overlay - added dragging logic for click versus drag
rebuilt min.js

@AndrewArace AndrewArace changed the title added maxZoom modifier overlay point support and other improvements and fixes Apr 24, 2015
Changed date-based id scheme to contain internal identifier to allow for
multiple instances of same layer types (multiple image layers, for
example)

Added visible() function to layer definition to change initial state of
layer visibility on stage load
@dciarletta
Copy link
Owner

Thank you for the contributions. I like all the changes you made. Although in the addition of the point support it appears you may have changed the display of the polygons when not in editMode. Originally if editMode is off the polygons would render without the points but I think that is not the case now. I apologize if I'm incorrect, but I didn't have time to test that theory out so I'm only going on code inspection at this point. If you don't get to it before I do I'll confirm if that is the case and patch it when I accept your pull request.

@AndrewArace
Copy link
Author

Hi,
Thanks for reviewing this - I did this in a relatively fast timeline for a small demo project, and figured I'd pass along some of my enhancements, but I admit that my changes may have impacted other functionality without meaning to.
I won't be working on this demo project any more (actually moving to a new company), so it's up to you if you believe the changes are worth me looking into and fixing, or if you'd rather just close the pull request without merging my changes. No hard feelings if you do.

@allnash
Copy link

allnash commented Feb 6, 2018

I have merged this to my fork. Thank. Sorry for chiming in 2018. 😆

@AndrewArace
Copy link
Author

AndrewArace commented Feb 6, 2018 via email

@allnash
Copy link

allnash commented Feb 12, 2018

@AndrewArace check this out. I redid parts of it remove layering. Created some dynamic components.

https://github.com/allnash/d3-v4-floorplan

@allnash
Copy link

allnash commented Feb 12, 2018

the stupid part was d3 v4 API differences.

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