-
Notifications
You must be signed in to change notification settings - Fork 157
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
Auto format trigger #2684
Auto format trigger #2684
Conversation
); | ||
if (formatOptions.changeSource) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What if instead of triggering another plugin event here we change the formatWithContentModel implementation by moving line 27
(Or at least the ChangeSource and ApiName properties destructing declaration) before line 81 but after calling the formatter callback
Since the option object is destructed to get the values before the callback and we modify the options object in the format callback the AutoFormat Change source is not correctly set.
@JiuqingSong to check if this is okay.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @BryanValverdeU!! That worked out better that my solution.
* Merge Link & Image Format when using MergeModel (#2681) * mergeLinkFormat * Also fix for images * remove unneeded changes * Remove more unneeded changes * Address comment * nit --------- Co-authored-by: Jiuqing Song <jisong@microsoft.com> * update versions * Merge pull request #2684 from microsoft/u/juliaroldi/trigger-auto-format Auto format trigger --------- Co-authored-by: Jiuqing Song <jisong@microsoft.com> Co-authored-by: Julia Roldi <87443959+juliaroldi@users.noreply.github.com>
Trigger event Auto format event in AutoFormatPlugin, so we can trigger the AutoFormat UI on OWA.