Moin
15 أغسطس 2024، 9:16ص
1
بغض النظر عما إذا كنت أنقر على “تأكيد” أو “تراجع”، فإن النص الموجود في المُنشئ هو “SuperTool script” بعد ذلك.
4 إعجابات
keegan
16 أغسطس 2024، 5:45م
3
شكراً @Moin على اكتشاف هذا! يجب أن يكون هذا قد تم حله الآن مع:
main ← fix-diff-modal
opened 11:36PM - 15 Aug 24 UTC
In https://github.com/discourse/discourse-ai/pull/715, we refactored the compose… r helper menu to use `DMenu` instead of `popperJS`. Since the `AiComposerHelperMenu` component was an instance of `DMenu`, having modals defined in the component's template directly was likely resulting in various issues. For example, clicking anywhere on the `DiffModal` was closing the modal which led to issues such as [being unable to revert.](https://meta.discourse.org/t/the-revert-button-in-the-changes-popup-does-not-work/321783/2)
In this PR we update the usage of the Diff and Thumbnail Suggestion modals to be instantiated by the modal service using `this.modal.show()` instead of being directly defined in the template. This allows for the modals to work correctly.
3 إعجابات
keegan
17 أغسطس 2024، 3:00م
5
تم إغلاق هذا الموضوع تلقائيًا بعد 21 ساعة. لم يعد يُسمح بالردود الجديدة.
