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

MarkdownPreview should scroll to top after selecting another note, also fix #3023 #3177

Merged
merged 2 commits into from
Aug 24, 2019

Conversation

hikerpig
Copy link
Contributor

@hikerpig hikerpig commented Jul 28, 2019

Description

This PR contains two commits modifying the MarkdownPreview component.

About the (not added) test

I tried to write some tests for MarkdownPreview component with jest, but got error from @rokt33r/js-sequence-diagrams package.

 FAIL  tests/components/MarkdownPreview.test.js
  ● Test suite failed to run

    TypeError: _.isEmpty is not a function

      at node_modules/@rokt33r/js-sequence-diagrams/dist/sequence-diagram-min.js:8:24486
      at Object.<anonymous> (node_modules/@rokt33r/js-sequence-diagrams/dist/sequence-diagram-min.js:8:25685)
      at Object.<anonymous> (browser/components/MarkdownPreview.js:47:27)
      at Object.<anonymous> (tests/components/MarkdownPreview.test.js:11:24)

Guess the test should be added after this is fixed.

Issue fixed

Type of changes

  • 🔘 Bug fix (Change that fixed an issue)
  • 🔘 Improvement (Change that improves the code. Maybe performance or development improvement)

Checklist:


IssueHunt Summary

Referenced issues

This pull request has been submitted to:


IssueHunt has been backed by the following sponsors. Become a sponsor

@hikerpig hikerpig changed the title Bugfix/scroll top error MarkdownPreview should scroll to top after selecting another note, also fix #3023 Jul 28, 2019
@Flexo013 Flexo013 added the awaiting review ❇️ Pull request is awaiting a review. label Jul 28, 2019
@Flexo013
Copy link
Contributor

This might also fix #2886 partially.

Copy link
Member

@ZeroX-DG ZeroX-DG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

@ZeroX-DG ZeroX-DG added needs extra review 🔎 Pull request requires review from an additional reviewer. and removed awaiting review ❇️ Pull request is awaiting a review. labels Aug 21, 2019
@Flexo013 Flexo013 requested a review from Rokt33r August 21, 2019 13:20
@Rokt33r Rokt33r added this to the v0.13.0 milestone Aug 24, 2019
@Rokt33r Rokt33r removed the needs extra review 🔎 Pull request requires review from an additional reviewer. label Aug 24, 2019
@Rokt33r Rokt33r merged commit 25ef456 into BoostIO:master Aug 24, 2019
@ZeroX-DG
Copy link
Member

@hikerpig I think your PR can also fix this issue, please also submit your PR there so that after we confirm with the issue author you can claim your reward:
https://issuehunt.io/r/BoostIO/Boostnote/issues/2038

@hikerpig
Copy link
Contributor Author

@ZeroX-DG Thanks a lot 🍻 .

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.

MD editor scrolls to the top whenever I add new content
4 participants