Greta

> works but this warning should be fixed > > _Property or method "onAccountUpdated" is not defined on the instance but referenced during render. Make sure that this property is...

fixed here: https://github.com/nextcloud/mail/pull/6998 because i cannot use account settings anymore and i need that for another pr

@jancborchardt and @nimishavijay please have a look at this solution and if you're ok with it

> * [ ] "Show image" → "Show images" > * [ ] The eye icon is not necessary, as per mockup > * [ ] In the menu, the...

> We can definitely add the option not to have an icon and have the menus triggered by tex-tonly buttons. I think it totally makes sense. i know that, but...

> How is this unblocked when it depends on [nextcloud/nextcloud-vue#2911](https://github.com/nextcloud/nextcloud-vue/pull/2911) and this app still uses v5? ah sorry didnt see you added the component on desc. I am not using...

not ready to merge

> not ready to merge ready now, please reviewers test it

> (Also, is there a specific reason we use a modal and not the same Viewer as in the Files app?) yes, attachments are saved in user folder which means...

Yes, its on our plan to work on this soon but no ETA yet