fix(ui): move blocked images notice out of the message body - #13463
fix(ui): move blocked images notice out of the message body#13463CreatorOfBiontium wants to merge 2 commits into
Conversation
The notice was inside the message's white body, where --color-text-maxcontrast is barely readable. Render it as its own section above the body, like the phishing warning, so it reads on the app background. Extract it into BlockedContentWarning and drop the :deep button overrides it needed, which also leaked onto the translate and read receipt buttons. AI-assisted: Claude Code (Claude Opus 5) Signed-off-by: Tobiáš Vašťák <tobias.vastak@gmail.com>
|
Thanks for the contribution |
|
Thanks a lot! Can you take a look at the conflicts? |
|
@GVodyanov do you think we should port that style to the translate hint? |
Ofc :) but I'm not home now, I'll do it on saturday |
The notice moved into its own component while main renamed the Actions components, added the iframe sandbox and swapped the translation check for browser-side language detection. Keep the extracted warning and take main's version of everything it sits next to. Assisted-by: ClaudeCode:claude-opus-5 Signed-off-by: Tobiáš Vašťák <tobias.vastak@gmail.com>
|
@kesselb conflicts solved |
|
Hello there, We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process. Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6 Thank you for contributing to Nextcloud and we hope to hear from you soon! (If you believe you should not receive this message, you can add yourself to the blocklist.) |
4 participants
Fixes #12075
Fixed the barely readable "The images have been blocked to protect your privacy." section.
The notice was inside the message's white body, where --color-text-maxcontrast was barely readable. Now we render it as its own section above the body, like the phishing warning, so it reads on the app background.
🤖 AI (if applicable)