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

[BUGFIX] Add safe checks to scroll logic #241

Merged
merged 1 commit into from
Oct 26, 2016
Merged

Conversation

offirgolan
Copy link
Collaborator

No description provided.

@offirgolan offirgolan merged commit f73cfa2 into master Oct 26, 2016
@offirgolan offirgolan deleted the scroll-safe-checks branch October 26, 2016 09:04
@buschtoens
Copy link
Collaborator

Just out of curiosity - what bug are you fixing here? 🤔

undefined instanceof Element and null instanceof Element both return false.

@offirgolan
Copy link
Collaborator Author

Haha not really a bug fix more of a precaution 😅

@offirgolan
Copy link
Collaborator Author

Actually now that I think about it... this change was pretty useless. Gonna go ahead and blame the late night on this one lol

@buschtoens
Copy link
Collaborator

Haha, thought so too! No Problem 😝

offirgolan added a commit that referenced this pull request Nov 7, 2016
* master: (42 commits)
  Resizable column fixes (#252)
  fix: repeated scrollToRow (#254)
  chore(package): update ember-cli-code-coverage to version 0.3.7 (#253)
  Demo Cleanup (#251)
  chore(package): update ember-lodash to version 0.0.11 (#245)
  chore(package): update ember-cli-code-coverage to version 0.3.6 (#249)
  Document `Row#content` (#250)
  Update resizable snippet
  Released v1.5.2
  [FEATURE] minResizeWidth + Event bubbling fix (#244)
  Update readme
  Update readme
  Released v1.5.1
  Add safe checks scroll logic (#241)
  [DOCFIX] Make breakpoints of demo app the same as ember-responsive's default (#240)
  Released v1.5.0
  Demo + Scrolling Cleanup (#238)
  [FEATURE] Responsive Columns (#235)
  [FEATURE] ScrollTo and ScrollToRow (#228)
  chore(package): update broccoli-asset-rev to version 2.5.0 (#227)
  ...
offirgolan added a commit that referenced this pull request Nov 14, 2016
* master: (24 commits)
  chore(package): update ember-wormhole to version 0.5.1 (#261)
  Update changelog
  Released v1.6.0
  [FEATURE] Draggable Columns (#258)
  chore(package): update ember-cli-code-coverage to version 0.3.8 (#257)
  Resizable column fixes (#252)
  fix: repeated scrollToRow (#254)
  chore(package): update ember-cli-code-coverage to version 0.3.7 (#253)
  Demo Cleanup (#251)
  chore(package): update ember-lodash to version 0.0.11 (#245)
  chore(package): update ember-cli-code-coverage to version 0.3.6 (#249)
  Document `Row#content` (#250)
  Update resizable snippet
  Released v1.5.2
  [FEATURE] minResizeWidth + Event bubbling fix (#244)
  Update readme
  Update readme
  Released v1.5.1
  Add safe checks scroll logic (#241)
  [DOCFIX] Make breakpoints of demo app the same as ember-responsive's default (#240)
  ...
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.

2 participants