AI-helper verbergt zich

keegan · 17 september 2024 om 00:38

This edge case issue should be now be resolved:

github.com/discourse/discourse-ai

FIX: Diff modal closing along with composer menu on mobile

discourse:main ← discourse:fix-edge-case

opened 08:20PM - 16 Sep 24 UTC

keegangeorge

+20 -5

## 🔍 Overview The `DiffModal` is triggered after selecting an option in the com…poser helper menu. After selecting an option, we should close the composer helper menu and only show the diff modal. On mobile, there was an edge-case where `this.args.close()` for was causing the closing of both the `DiffModal` and the `AiComposerHelperMenu`. This PR resolves that by ensuring the menu is closed _first_ asynchronously, followed by opening the relevant modal. ## 📹 Preview **Before** https://github.com/user-attachments/assets/8f7c0cf0-864a-4875-ade2-be44dc0b5a7c **After** https://github.com/user-attachments/assets/b91f8999-4981-4da3-9449-f163245f1812

@Jagster can you confirm its resolved for you on your iPad?

Topic		Antwoorden	Weergaven
AI helper has disappeared in composer Bug discourse-hub , ai , ai-helper , fixed	9	113	3 september 2024
AI helper is hiding. Again Bug ai , ai-helper	4	66	23 januari 2025
Android context menu covers up the composer AI helper Bug composer , mobile , ai , ai-helper , fixed	31	196	24 augustus 2024
AI helper hiding on Android tablets Bug ai , ai-helper , tablet	7	92	31 augustus 2025
Discourse composer and Firefox on Android tablet UX tablet	14	165	19 april 2025

AI-helper verbergt zich

Gerelateerde topics