Skip to content
This repository was archived by the owner on May 7, 2021. It is now read-only.

fix(markdown): emit event on outside-click if no data changed #157

Merged
merged 2 commits into from
Sep 27, 2018

Conversation

divyanshiGupta
Copy link
Contributor

Fixes: openshiftio/openshift.io#4049

This PR is a part of this PR fabric8-ui/fabric8-planner#2781 in fabric8-ui

@centos-ci
Copy link
Collaborator

Can one of the admins verify this patch?

@divyanshiGupta
Copy link
Contributor Author

@sahil143 ^^

@divyanshiGupta
Copy link
Contributor Author

@sahil143 can you take a look at this PR?

@joshuawilson
Copy link
Contributor

Once this is merged, it can only be applied to the planner upgrade PR/branch as it has already been upgraded.

@joshuawilson
Copy link
Contributor

Please update the tests.
[test]

@divyanshiGupta
Copy link
Contributor Author

@joshuawilson done

const clickedInside = this.elementRef.nativeElement.contains(target);

if (this.editorInput) {
this.isNoDataChanged = this.editorInput.nativeElement.innerText.trim() == this.rawText;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use === here

@divyanshiGupta
Copy link
Contributor Author

@sahil143 done

@joshuawilson joshuawilson merged commit b4b77a8 into fabric8-ui:master Sep 27, 2018
@fabric8cd
Copy link

🎉 This PR is included in version 3.1.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants