Skip to content

Move drag/drop into separate file, clean up row, move detectIE/detectChrome from perspective into perspective-viewer#293

Merged
texodus merged 1 commit intomasterfrom
row_dragdrop_refactor
Nov 7, 2018
Merged

Move drag/drop into separate file, clean up row, move detectIE/detectChrome from perspective into perspective-viewer#293
texodus merged 1 commit intomasterfrom
row_dragdrop_refactor

Conversation

@sc1f
Copy link
Contributor

@sc1f sc1f commented Nov 5, 2018

  • drag/drop handlers no longer inlined inside view.js
  • row uses register_id and register_callback pattern
  • detectIE and detectChrome moved into perspective-viewer; I couldn't find anywhere in perspective engine that called it, and since the calls were happening in viewer and highcharts, I moved it into the utils of the viewer.

I would have split this into two different branches with different PRs, but the changes were initially made together, making resetting and splitting into different branches difficult.

@sc1f sc1f changed the title Move drag/drop into separate file, clean up row Move drag/drop into separate file, clean up row, move detectIE/detectChrome from perspective into perspective-viewer Nov 5, 2018
of view.js and into separate file, change dragdrop handlers to
underscore_naming
@texodus texodus force-pushed the row_dragdrop_refactor branch from 584f89c to 42612d8 Compare November 7, 2018 19:27
@texodus texodus merged commit 53b30df into master Nov 7, 2018
@texodus texodus deleted the row_dragdrop_refactor branch November 7, 2018 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants