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

fix: wrong popup in markdown preview (close #210) #211

Merged
merged 2 commits into from
Jun 18, 2018

Conversation

kyuwoo-choi
Copy link
Member

Please check if the PR fulfills these requirements

  • It's the right issue type on the title
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes/features)
  • Docs have been added/updated (for bug fixes/features)
  • It does not introduce a breaking change or has a description of the breaking change

Description

#210
Enabling/Disabling button feature is added to resolve the issue.


Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

@kyuwoo-choi kyuwoo-choi force-pushed the fix/wrong_popup_in_markdown_preview branch from 6f9223a to ef3fcc4 Compare June 17, 2018 07:29
@@ -255,6 +257,8 @@ class EventManager {
if (handler.namespace !== namespace) {
handlersToSurvive.push(handler);
}

return null;
Copy link
Member

Choose a reason for hiding this comment

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

주석 변경되어야겠네요~ (프라이빗이라 별 의미없어보이지만요 ;ㅅ;)
{?string}

Copy link
Member Author

Choose a reason for hiding this comment

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

내부의 애로우 함수라 주석과 무관합니다.

Copy link
Member

Choose a reason for hiding this comment

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

엇 그렇네요. 접혀져 있어서 잘못봤네요 ㅠㅜ

@seonim-ryu
Copy link
Member

[6/18] 리뷰 완료하였습니다.

@jinwoo-kim-nhn
Copy link

리뷰 완료하였습니다. 수고하셨습니다.

@kyuwoo-choi kyuwoo-choi merged commit edec8a7 into master Jun 18, 2018
@kyuwoo-choi kyuwoo-choi deleted the fix/wrong_popup_in_markdown_preview branch June 18, 2018 05:58
seonim-ryu pushed a commit that referenced this pull request Jan 2, 2020
* feat: button enable/disable

* fix: wrong popup in markdown preview
seonim-ryu pushed a commit that referenced this pull request Jan 6, 2020
* feat: button enable/disable

* fix: wrong popup in markdown preview
seonim-ryu pushed a commit that referenced this pull request Feb 5, 2020
seonim-ryu pushed a commit that referenced this pull request Feb 5, 2020
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.

None yet

3 participants