Feature/message dialog flags #1
Conversation
|
Ik kan de optie om een bericht te flaggen niet vinden. Waar kan ik dit doen? |
|
Wanneer ik de repo clone en enable staat de flag feature onder een DM bericht. Het is niet heel goed verwerkt, maar wel beschikbaar:
|
|
@julianW05 @luceos De flag knop is on hover van de Post-Actions (in mijn andere thema-PR heb ik het rechts van het bericht gestyled). Zonder thema, waar zal ik deze standaard plaatsen voor deze PR? We kunnen de rapporteerknop ook in de moderator opties dropdown plaatsen (de '···' moderatie dropdown) https://github.com/flarum/framework/blob/2.x/extensions/messages/js/src/forum/utils/MessageControls.tsx . Alleen is deze dropdown volgens mij niet ge-export binnen de Messages extensie zelf (überhaupt niet beschikbaar), ik kon er in ieder geval op geen enkele manier op extenden toen ik dit probeerde. De translations heb ik nu ook aangepast en instellingen zijn nu ook geüpdatet waardoor je de instellingen van de extensie nu normaal kan gebruiken zonder dat deze de flag-extensie ook aanpast. Ik ben nu bezig met het beschikbaar maken van het gehele gesprek wanneer deze gerapporteerd is (of willen we slechts het gerapporteerde bericht zonder context / een x aantal berichten laten zien?). |
Nee de flagging moet meteen de hele thread beschikbaar maken voor diegene met moderation permissions daarover.
Ik denk dat het geen probleem is om de knop alleenstaand te houden in de body, on hover, mits dit goed gestyled is. Echter is er een probleem met mobile, daar werkt hover niet. Dus als een control dropdown noodzakelijk is, dan moeten we die beschikbaar maken. Als een PR naar framework nodig is voor de export van dit component dan graag zeer binnenkort uitvoeren zodat dit voor de release van 2.0 nog gemerged kan worden (graag mij mentionen als je dit doet). Qua UX laat ik de keuze bij jou @jaspervriends |
|
Zal ik het mergen @luceos ? |
|
@julianW05 ja. Tnx. |
No description provided.